[wp-trac] [WordPress Trac] #24988: Cleanup for `/wp-admin/js/accordion.js`
WordPress Trac
noreply at wordpress.org
Thu Aug 8 05:55:07 UTC 2013
#24988: Cleanup for `/wp-admin/js/accordion.js`
--------------------------+-----------------------------
Reporter: carldanley | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version: trunk
Severity: normal | Keywords:
--------------------------+-----------------------------
I've attached a cleanup iteration of the existing accordion file.
* Cleaner function names
* Moved the structure of the file to a revealing module pattern
* Added JSDocs
* prefixed jQuery objects with proper `$` sign
* The current iteration has no publicly exposed API methods but I was
thinking it might be nice to offer support for `toggleAccordion` etc. Need
thoughts on this.
Could use a second pair of eyes to test this functionality and make sure
we have no errors here.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24988>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list