[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