> if (this.options.returnHeightToAuto) {
>  var el = this.elements[index];
>  el.setStyle('height', 'auto');
> };

> Needs to be changed to:

> if (this.options.returnHeightToAuto && index >= 0) {
>  var el = this.elements[index];
>  el.setStyle('height', 'auto');
> };

> On 21 Oct 2009, at 15:00, Jeremy B wrote:

> I am getting an error when trying to create a nested accordion and trying
> to make it so you can close all of the accordion panels.

> I have posted a sample on mooshell:
> http://mooshell.net/kxYpt/

> This is the error I am getting in firebug:

> el is undefined
> file:///mootools-1.2.4.1-more.js
> Line 221

> This is line 221 in my mootoos-more.js file
> 221 el.setStyle('height', 'auto');

> It seems like it may be a bug, but I am not entirely sure, perhaps is it
> due to me trying to nest accordions?
> Any help is greatly appreciated,
> Jeremy

> On Fri, Oct 23, 2009 at 7:44 AM, Oskar Krawczyk <oskar.krawc...@gmail.com
> > wrote:
> Yup, a bug inside the .display method:

> if (this.options.returnHeightToAuto) {
>    var el = this.elements[index];
>    el.setStyle('height', 'auto');
> };

> Needs to be changed to:

> if (this.options.returnHeightToAuto && index >= 0) {
>    var el = this.elements[index];
>    el.setStyle('height', 'auto');
> };

> On 21 Oct 2009, at 15:00, Jeremy B wrote:

>> I am getting an error when trying to create a nested accordion and  
>> trying to make it so you can close all of the accordion panels.

>> I have posted a sample on mooshell:
>> http://mooshell.net/kxYpt/

>> This is the error I am getting in firebug:

>> el is undefined
>> file:///mootools-1.2.4.1-more.js
>> Line 221

>> This is line 221 in my mootoos-more.js file
>> 221 el.setStyle('height', 'auto');

>> It seems like it may be a bug, but I am not entirely sure, perhaps  
>> is it due to me trying to nest accordions?
>> Any help is greatly appreciated,
>> Jeremy

> On 23 Oct 2009, at 20:45, Aaron Newton wrote:

> will fix for next release. hopefully monday.

> On Fri, Oct 23, 2009 at 7:44 AM, Oskar Krawczyk <oskar.krawc...@gmail.com>wrote:

>> Yup, a bug inside the .display method:

>> if (this.options.returnHeightToAuto) {
>>  var el = this.elements[index];
>>  el.setStyle('height', 'auto');
>> };

>> Needs to be changed to:

>> if (this.options.returnHeightToAuto && index >= 0) {
>>  var el = this.elements[index];
>>  el.setStyle('height', 'auto');
>> };

>> On 21 Oct 2009, at 15:00, Jeremy B wrote:

>> I am getting an error when trying to create a nested accordion and trying
>> to make it so you can close all of the accordion panels.

>> I have posted a sample on mooshell:
>> http://mooshell.net/kxYpt/

>> This is the error I am getting in firebug:

>> el is undefined
>> file:///mootools-1.2.4.1-more.js
>> Line 221

>> This is line 221 in my mootoos-more.js file
>> 221 el.setStyle('height', 'auto');

>> It seems like it may be a bug, but I am not entirely sure, perhaps is it
>> due to me trying to nest accordions?
>> Any help is greatly appreciated,
>> Jeremy

> It's odd to me that after some extensive re-facotring of the Accordion
> class, no one actually tested it thoroughly.

> On 24 Oct 2009, at 18:08, davidck wrote:

> Jeremy, I noticed the issue you pointed out as well.

> I fixed it by setting the new option returnHeightToAuto to true or
> false. Forgot which one, basically the opposite of the default boolean
> value.

> On Oct 24, 11:34 am, Jeremy B <jbue...@gmail.com> wrote:

> Yeah I put the fix into my mootools-more and the JS error is gone, but the

> accordion does snap back open after it closes.

> Jeremy Buelerwww.jbueler.com

> On Sat, Oct 24, 2009 at 5:19 AM, soundseller <soundsel...@googlemail.com
> >wrote:

> @oscar I tried your fix but

>  it still does not work as expected.

> If I try to close all elements, the accordion snaps open again.

> Any ideas?

> I use following code:

> var accordion = new Accordion($$('.toggler'),$$('.element'), {

>                opacity: 0,

>                alwaysHide: true,

>                onActive: function(toggler) { toggler.setStyle('color',

> '#000000'); },

>                onBackground: function(toggler) { toggler.setStyle('color',

> '#666666'); }

>        });

> On Oct 24, 3:15 am, "Steve Onnis" <st...@cfcentral.com.au> wrote:

> I use

> display : null

> -----Original Message-----

> From: Piotr Zalewa [mailto:zal...@gmail.com]

> Sent: Saturday, 24 October 2009 12:32 AM

> To: mootools-users@googlegroups.com

> Subject: [Moo] Re: Fx.accordionError When using alwaysHide: true

> code simplified a bit

> the problem is with the display: -1

> don't even know if it is the right usage - it's working if commented

> outhttp://mooshell.net/kxYpt/1

> zalun

> --

> London, UK