[wp-trac] [WordPress Trac] #16695: quicktags needs some refactoring

WordPress Trac wp-trac at lists.automattic.com
Sun Mar 27 05:32:37 UTC 2011


#16695: quicktags needs some refactoring
----------------------------------------------+----------------------------
 Reporter:  garyc40                           |       Owner:  scribu
     Type:  enhancement                       |      Status:  reviewing
 Priority:  normal                            |   Milestone:  Awaiting
Component:  Editor                            |  Review
 Severity:  normal                            |     Version:  3.1
 Keywords:  has-patch dev-feedback 3.2-early  |  Resolution:
----------------------------------------------+----------------------------

Comment (by garyc40):

 Replying to [comment:15 scribu]:
 > Ok, well you din't mention that anywhere. :)

 Sorry for the confusion. Just attached a patch that restores the class
 name to `QTags` instead of `QuickTags`, and add 2 globals variables
 `edButtons` as well as `edButton` that maps nicely to the new API.
 [http://scribu.net/wordpress/right-way-to-add-custom-quicktags.html The
 code sample that you linked] to works perfectly fine with the new API now.

 Hopefully the outcome of this ticket will allow for a more formal quicktag
 API than what we have now. I'm willing to further update this ticket to
 comply with core commiters' direction.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16695#comment:16>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list