This project is archived and is in readonly mode.

#179 ✓resolved
batman42ca

Accordion constructor documentation

Reported by batman42ca | November 8th, 2009 @ 03:26 PM | in 1.2.4.3

The accordion docs would seem to be incorrect and actually aren't even consistent within themselves. It shows 3 parameters (togglers, elements, options) in the constructor, then goes on to explain 4 parameters (togglers, elements, container, options) in the wrong order (container should be first) according to the working demo

http://demos.mootools.net/Accordion

... which I've tested using mootools more 1.2.4.2

Comments and changes to this ticket

  • batman42ca

    batman42ca November 8th, 2009 @ 04:11 PM

    I just attempted to make this correction myself through GitHub. I hope I did it right.

  • Aaron Newton

    Aaron Newton November 19th, 2009 @ 02:08 AM

    • State changed from “new” to “resolved”
    • Milestone set to 1.2.4.3
    • Assigned user set to “Aaron Newton”

    actually, I've removed all the references to container; it's a legacy thing that is no longer used at all.

  • batman42ca

    batman42ca November 19th, 2009 @ 11:57 AM

    • Tag changed from 1.2.4.2, accordion, documentation to 1, accordion, documentation

    Won't that be a breaking change? If the first parameter in my code was "container" and you remove container as a parameter, legacy code will fail.

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

The MooTools Extensions

People watching this ticket

Pages