add specs, add docs and fix to keepOpen option

Author: SergioCrisostomo

Docs/Fx/Fx.Accordion.md

@@ -42,6 +42,7 @@ Fx.Accordion Method: constructor
 * alwaysHide  - (*boolean*: defaults to false) If set to true, it will be possible to close all displayable elements. Otherwise, one will remain open at all time.
 * initialDisplayFx - (*boolean*; defaults to true) If set to false, the initial item displayed will not display with an effect but will just be shown immediately.
 * resetHeight - (*boolean*; defaults to true) If set to false, the height of an opened accordion section will be set to an absolute pixel size.
+* keepOpen - (*boolean*; defaults to false) If set to true, it will keep multiple elements open (no auto collapsing). To close a element you have to "click" the trigger again.
 
 ### Events
 

Source/Fx/Fx.Accordion.js

@@ -161,11 +161,13 @@ Fx.Accordion = new Class({
 		var obj = {},
 			elements = this.elements,
 			options = this.options,
-			effects = this.effects;
+			effects = this.effects,
+			keepOpen = options.keepOpen,
+			alwaysHide = options.alwaysHide;
 
 		if (useFx == null) useFx = true;
 		if (typeOf(index) == 'element') index = elements.indexOf(index);
-		if (index == this.current && !options.alwaysHide) return this;
+		if (index == this.current && !alwaysHide && !keepOpen) return this;
 
 		if (options.resetHeight){
 			var prev = elements[this.current];
@@ -174,19 +176,21 @@ Fx.Accordion = new Class({
 			}
 		}
 
-		if ((this.timer && options.link == 'chain') || (index === this.current && !options.alwaysHide)) return this;
+		if (this.timer && options.link == 'chain') return this;
 
 		if (this.current != null) this.previous = this.current;
 		this.current = index;
 		this.selfHidden = false;
 
 		elements.each(function(el, i){
 			obj[i] = {};
-			var hide;
-			if(!options.keepOpen || i == index){
+			var hide, isOpen;
+			if(!keepOpen || i == index){
+	            if (i == index) isOpen = (el.offsetHeight > 0 && options.height) || (el.offsetWidth > 0 && options.width);
+			
 				if (i != index){
 					hide = true;
-				} else if (options.alwaysHide && ((el.offsetHeight > 0 && options.height) || el.offsetWidth > 0 && options.width)){
+				} else if ((alwaysHide || keepOpen) && isOpen){
 					hide = true;
 					this.selfHidden = true;
 				}

Tests/Interactive/Fx/Fx.Accordion.html

@@ -150,3 +150,24 @@
 });
 </script>
 
+<hr/>
+
+<p>It should keep more than one section open at the time.</p>
+
+<dl id="acc7">
+  <dl>
+    <dt class="toggle"><b>first section</b></dt>
+    <dd class="stretcher">I'm the content for the first section.</dd>
+    <dt class="toggle"><b>second section</b></dt>
+    <dd class="stretcher">I'm the content for the second section.</dd>
+    <dt class="toggle"><b>third section</b></dt>
+    <dd class="stretcher">I'm the content for the third section.</dd>
+  </dl>
+</dl>
+
+<script>
+new Fx.Accordion($$('#acc7 dt'), $$('#acc7 dd'), {
+	keepOpen: true
+});
+</script>
+