WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#15206 closed defect (bug) (fixed)

Add class to 'comment awaiting moderation' element

Reported by: 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 4 years ago.

Download all attachments as: .zip

Change History (7)

@nacin4 years ago

comment:1 @sbressler4 years ago

  • Cc sbressler@… added

comment:2 @jane4 years ago

  • Keywords has-patch added

comment:3 @westi4 years ago

  • Keywords dev-reviewed added

Looks good to me.

comment:4 @iandstewart4 years ago

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

comment:5 @nacin4 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.

comment:6 @automattor4 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.