http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90729

Revision: 90729
Author:   krinkle
Date:     2011-06-24 19:52:35 +0000 (Fri, 24 Jun 2011)
Log Message:
-----------
(bug 29567) mw.util.addPortletLink should only wrap link in <span> for 
"vectorTabs" portlets
* Patch by Erwin Dokter
* This unbreaks test introduced in r90728

Modified Paths:
--------------
    trunk/phase3/resources/mediawiki/mediawiki.util.js

Modified: trunk/phase3/resources/mediawiki/mediawiki.util.js
===================================================================
--- trunk/phase3/resources/mediawiki/mediawiki.util.js  2011-06-24 19:50:53 UTC 
(rev 90728)
+++ trunk/phase3/resources/mediawiki/mediawiki.util.js  2011-06-24 19:52:35 UTC 
(rev 90729)
@@ -374,9 +374,14 @@
                                // Unhide portlet if it was hidden before
                                $portlet.removeClass( 'emptyPortlet' );
 
-                               // Wrap the anchor tag in a <span> and create a 
list item for it
+                               // Wrap the anchor tag in a list item (and a 
span if $portlet is a Vector tab)
                                // and back up the selector to the list item
-                               var $item = $link.wrap( 
'<li><span></span></li>' ).parent().parent();
+                               var $item;
+                               if ( $portlet.hasClass( 'vectorTabs' ) ) {
+                                       $item = $link.wrap( 
'<li><span></span></li>' ).parent().parent();
+                               } else {
+                                       $item = $link.wrap( '<li></li>' 
).parent();
+                               }
 
                                // Implement the properties passed to the 
function
                                if ( id ) {


_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to