[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