From 8fad46e6add5d9de10606e8acf519d443ca85848 Mon Sep 17 00:00:00 2001
From: Krinkle <krinkle@users.mediawiki.org>
Date: Sun, 2 Jan 2011 04:46:44 +0000
Subject: [PATCH] * Toggle entire element if no .mw-collapsible-content is
 found * Bug fixed in which a single customtoggle should trigger multiple
 customcollapsible elements

If no .mw-collapsible-content element is found when the toggleElement() function is called, assume that the user wants the mw-collapsible itself to be toggled. This enables custom use with table rows (in which we dont want something inside to toggle but the entire table row) - with a remote toggle.

This change makes it possible to have a togglable table-row.

The intial "collapsed" state for a mw-customtoggle is no longer executed by performing a click on the mw-customtoggle. Instead by calling the toggle function directly.
Reason being that an mw-customtoggle could have any number of special functions bound to it (be it the function to toggle another mw-customcollapsible, or perhaps a totally different function).

I considered using this method for 'normal' default togglers as well. However default togglers have more actions involved aside from just 'toggling' the element. Namely changing the innerText to the localized message and swapping classes. Those stay as they are by simply simulating a click() event on them (if the initial state should be collapsed).
---
 resources/jquery/jquery.makeCollapsible.js | 69 +++++++++++++++-------
 1 file changed, 47 insertions(+), 22 deletions(-)

diff --git a/resources/jquery/jquery.makeCollapsible.js b/resources/jquery/jquery.makeCollapsible.js
index 887e468d46..127cc8e6ca 100644
--- a/resources/jquery/jquery.makeCollapsible.js
+++ b/resources/jquery/jquery.makeCollapsible.js
@@ -63,7 +63,21 @@ $.fn.makeCollapsible = function() {
 						}
 	
 					} else { // <div>, <p> etc.
-						$collapsible.find( '> .mw-collapsible-content' ).slideUp();
+						var $collapsibleContent = $collapsible.find( '> .mw-collapsible-content' );
+						
+						// If a collapsible-content is defined, collapse it
+						if ( $collapsibleContent.size() ) {
+							$collapsibleContent.slideUp();
+
+						// Otherwise assume this is a customcollapse with a remote toggle
+						// .. and there is no collapsible-content because the entire element should be toggled
+						} else {
+							if ( $collapsible.is( 'tr' ) || $collapsible.is( 'td' ) || $collapsible.is( 'th' ) ) {
+								$collapsible.fadeOut();
+							} else {
+								$collapsible.slideUp();
+							}
+						}
 					}
 
 				} else {
@@ -86,7 +100,21 @@ $.fn.makeCollapsible = function() {
 						}
 	
 					} else { // <div>, <p> etc.
-						$collapsible.find( '> .mw-collapsible-content' ).slideDown();
+						var $collapsibleContent = $collapsible.find( '> .mw-collapsible-content' );
+						
+						// If a collapsible-content is defined, collapse it
+						if ( $collapsibleContent.size() ) {
+							$collapsibleContent.slideDown();
+
+						// Otherwise assume this is a customcollapse with a remote toggle
+						// .. and there is no collapsible-content because the entire element should be toggled
+						} else {
+							if ( $collapsible.is( 'tr' ) || $collapsible.is( 'td' ) || $collapsible.is( 'th' ) ) {
+								$collapsible.fadeIn();
+							} else {
+								$collapsible.slideDown();
+							}
+						}
 					}
 				}
 			},
@@ -145,21 +173,16 @@ $.fn.makeCollapsible = function() {
 				return;
 			},
 			// Toggles customcollapsible
-			toggleLinkCustom = function( that, e ) {
-				var	$that = $(that),
-					classes = that.className.split(' ');
-				e.preventDefault();
-				// Check each class to see if it belongs to a customcollapse
-				for ( i = 0; i < classes.length; i++ ) {
-					if ( classes[i].indexOf( 'mw-customtoggle-' ) === 0 ) {
-						var id = '#' + classes[i].replace( 'mw-customtoggle-', 'mw-customcollapsible-' ),
-							$collapsible = $( id ),
-							action = $collapsible.hasClass( 'mw-collapsed' ) ? 'expand' : 'collapse';
-
-						$collapsible.toggleClass( 'mw-collapsed' );
-						toggleElement( $collapsible, action, $that );
-					}
+			toggleLinkCustom = function( $that, e, $collapsible ) {
+				// For the initial state call of customtogglers there is no event passed
+				if (e) {
+					e.preventDefault();
 				}
+				// Get current state and toggle to the opposite
+				var action = $collapsible.hasClass( 'mw-collapsed' ) ? 'expand' : 'collapse';
+				$collapsible.toggleClass( 'mw-collapsed' );
+				toggleElement( $collapsible, action, $that )
+				
 			};
 
 		// Use custom text or default ?
@@ -194,15 +217,17 @@ $.fn.makeCollapsible = function() {
 			// Double check that there is actually a customtoggle link
 			if ( $customTogglers.size() ) {
 				$customTogglers.bind( 'click.mw-collapse', function( e ) {
-					toggleLinkCustom( this, e );
+					toggleLinkCustom( $(this), e, $that );
 				} );
 			} else {
 				mw.log( '#' + thatId + ': Missing toggler!' );
 			}
 			
-			// To change initial state at the bottom of the script
-			// Set this variable to one of the togglers
-			var $toggleLink = $customTogglers.eq(0);
+			// Initial state
+			if ( $that.hasClass( 'mw-collapsed' ) ) {
+				$that.removeClass( 'mw-collapsed' );
+				toggleLinkCustom( $customTogglers, null, $that )
+			}
 
 		// If this is not a custom case, do the default:
 		// Wrap the contents add the toggle link 
@@ -262,8 +287,8 @@ $.fn.makeCollapsible = function() {
 			}
 		}
 
-		// Initial state
-		if ( $that.hasClass( 'mw-collapsed' ) ) {
+		// Initial state (only for those that are not custom)
+		if ( $that.hasClass( 'mw-collapsed' ) && $that.attr( 'id' ).indexOf( 'mw-customcollapsible-' ) !== 0 ) {
 			$that.removeClass( 'mw-collapsed' );
 			$toggleLink.click();
 		}
-- 
2.20.1