Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#15206 closed defect (bug) (fixed)

Add class to 'comment awaiting moderation' element

Reported by: nacin's profile nacin Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-patch dev-reviewed
Focuses: Cc:

Description

On my site, I needed to override twentyten_comment() just so I could effectively target the <em> element that contains "Your comment is awaiting moderation."

This is derived directly from Walker_Comment.

I wished we could wrap that in a <div> without screwing up a lot of styling, given that it is generally styled as a de-facto block-level element -- indeed, it is followed immediately by an unsemantic <br/> -- but maybe we can just add a class to it?

Patch attached that handles both Walker_Comment and twentyten_comment.

Attachments (1)

15206.diff (1.3 KB) - added by nacin 14 years ago.

Download all attachments as: .zip

Change History (7)

@nacin
14 years ago

#1 @sbressler
14 years ago

  • Cc sbressler@… added

#2 @jane
14 years ago

  • Keywords has-patch added

#3 @westi
14 years ago

  • Keywords dev-reviewed added

Looks good to me.

#4 @iandstewart
14 years ago

Adding a class makes the most sense to me. Looks good.

#5 @nacin
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [16028]) Add a class to 'Your comment is awaiting moderation' in the comment walker and Twenty Ten's callback. fixes #15206.

#6 @automattor
13 years ago

(In [17739]) More Twenty Eleven cleanup and add the comment-awaiting-moderation class. see #17198. see #15206 for history on the CSS class.

Note: See TracTickets for help on using tickets.