Opened 2 years ago

Last modified 2 years ago

#16944 new enhancement

Add new 'bynetworkuser' CSS class to output from comment-template.php

Reported by: jacques_chester Owned by:
Priority: normal Milestone: Awaiting Review
Component: Comments Version:
Severity: normal Keywords: has-patch
Cc:

Description

Currently, the comment-template.php file checks to see whether a comment author is in the wp_user table.

If they are, then WordPress will emit a 'byuser' class as part of the comment div markup.

However, this behaviour is not ideal from multisite installations. 'byuser' should be restricted on a per-blog basis.

I propose that the current behaviour be assigned to a new CSS class, 'bynetworkuser'.

Attachments (2)

byuser.diff (1.1 KB) - added by jacques_chester 2 years ago.
Preliminary diff
16944.diff (1.1 KB) - added by solarissmoke 2 years ago.

Download all attachments as: .zip

Change History (6)

Preliminary diff

Your diff doesn't contain the file name (it's just ".php").

Please make patches against the root wp directory.

comment:2 follow-up: ↓ 3   solarissmoke2 years ago

Is changing the context in which the byuser class is applied going to cause back-compat issues?

comment:3 in reply to: ↑ 2   nacin2 years ago

Replying to solarissmoke:

Is changing the context in which the byuser class is applied going to cause back-compat issues?

Yes.

If anything we should add by-user-of-blog.

  • Keywords has-patch added

If we want to add this, here's a patch, otherwise wontfix.

Note: See TracTickets for help on using tickets.