[wp-trac] Re: [WordPress Trac] #7435: Reply to comments from admin

WordPress Trac wp-trac at lists.automattic.com
Sat Aug 30 09:31:02 GMT 2008


#7435: Reply to comments from admin
----------------------------+-----------------------------------------------
 Reporter:  ryan            |        Owner:  azaozz
     Type:  enhancement     |       Status:  new   
 Priority:  normal          |    Milestone:  2.7   
Component:  Administration  |      Version:        
 Severity:  normal          |   Resolution:        
 Keywords:  blessed         |  
----------------------------+-----------------------------------------------
Comment (by caesarsgrunt):

 Replying to [comment:9 nbachiyski]:
 > Here is a little patch, which hides the Reply button if we don't have
 JS. Also, fixes some small HTML validations issues.

 I don't like the nested spans :
 {{{
 <span class="reply"><span class="hide-if-no-js">...</span></span>
 }}}
 We could get one span only by doing
 {{{
 $actions['reply hide-if-no-js'] = ...
 }}}
 but that wouldn't be good for plugins etc which want to modify the reply
 link.

 Attached is a patch which allows any of the actions links to specify
 additional class names by optionally being arrays instead of
 strings.[[BR]]
 This also enhances the extensibility of the system by allowing plugins to
 specify additional classes for action-links which they add.

 What does anyone else think of this? Is there anything wrong with this
 system? Have I broken anything? :)

-- 
Ticket URL: <http://trac.wordpress.org/ticket/7435#comment:17>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list