dangit. I didn't see it. I just implemented it. I'll resolve it. On Fri, Oct 23, 2009 at 12:51 PM, Oskar Krawczyk <[email protected]>wrote:
> I've already requested a pull with the fix... > > 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 > <[email protected]>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 >> >> >> > >
