[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