This project is archived and is in readonly mode.

#179 ✓resolved

Accordion constructor documentation

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

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

... which I've tested using mootools more

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
    • 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, 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