Twenty Thirteen: Highlight post author comments

[johnbillion]

Previous bundled themes highlight comments by the current post author. Twenty Thirteen should do the same.

[johnbillion]

Screenshot of Twenty Twelve

[johnbillion]

Note that highlighting of post author comments is currently broken in trunk due to r23353. See #22324.

[lancewillett]

Do you mean "Highlight post author comments *better*"? They are already styled with a star in front of the username: ​http://cl.ly/image/1B0h3L161b1k

[johnbillion]

Okay yeah I do mean that. I hadn't even spotted the star. I personally like the "Post author" message that Twenty Twelve adds.

[lancewillett]

Possible author text label

[lancewillett]

Added a patch with added text label; patch includes styles from @joen.

[SergeyBiryukov]

Would it make sense to use get_post() instead of the global (for consistency with [21735])?

[nacin]

I am not a huge fan of 23522.patch​ because it makes it more difficult to adopt the standard wp_list_comments() comment callback.

Yes, get_post() makes more sense, but since $comment is actually passed to the callback, I'd avoid using any global state if possible.

[lancewillett]

I agree with Nacin -- we should favor core HTML5 output over custom theme-specific output.

If we can get #20088 into 3.6 we can close this ticket "wontfix".

[lancewillett]

I looped back with Joen, and he was clear that just the star was the intention of the design to indicate the post author.