Catrope has submitted this change and it was merged.
Change subject: Fix occasional collapsibleTabs JS error
......................................................................
Fix occasional collapsibleTabs JS error
vector/vector.js depends on $.fn.collapsibleTabs, but that's defined in
vector/collapsibleTabs.js which is loaded *after*. The only reason it
worked most of the time is because the invocation is wrapped in a document
ready callback. Most of the time, the document isn't ready yet when
vector.js runs, so the callback is deferred and $.fn.collapsibleTabs is
created before the callback runs. But if the document is already ready,
the callback runs immediately without $.fn.collapsibleTabs having been
created yet, and so it throws an error.
Fixed by including the .js files in the proper order.
Change-Id: If101efe82970bda7a39d827ce3c668abad6a5f5e
(cherry picked from commit e25445634ea4045f9945e892e6e2df389cb6f1c9)
(cherry picked from commit 644087d88cff8f1ad49b627dba85d094a05fe77e)
---
M resources/Resources.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Catrope: Verified; Looks good to me, approved
diff --git a/resources/Resources.php b/resources/Resources.php
index fcb3d6a..35c2253 100644
--- a/resources/Resources.php
+++ b/resources/Resources.php
@@ -100,8 +100,8 @@
),
'skins.vector.js' => array(
'scripts' => array(
- 'vector/vector.js',
'vector/collapsibleTabs.js',
+ 'vector/vector.js',
),
'position' => 'top',
'dependencies' => 'jquery.delayedBind',
--
To view, visit https://gerrit.wikimedia.org/r/71951
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If101efe82970bda7a39d827ce3c668abad6a5f5e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.22wmf8
Gerrit-Owner: Catrope <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
