Opened 23 months ago

Closed 8 months ago

#17978 closed defect (bug) (fixed)

Label for "Super Admin" in wp-admin/user-edit.php seems redundant and inconsistent

Reported by: waclawjacek Owned by: ryan
Priority: normal Milestone: 3.5
Component: Network Admin Version: 3.2
Severity: trivial Keywords: has-patch commit
Cc:

Description

Hello :-)

Please excuse me for the harsh words. Explanation coming right up.

The following is the relevant bit of code:

<tr><th><label for="role"><?php _e('Super Admin'); ?></label></th>

(file name wp-admin/user-edit.php, line number in revision 18392: 263).

The label tag seems to be both redundant and inconsistent - redundant, because there's no input field that has the ID "role" on that screen, and inconsistent, because all the other checkbox groups don't have any labels (as in "HTML <label> tags") to the left of them.

Cheers!
WJ

Attachments (1)

patch.diff (793 bytes) - added by waclawjacek 23 months ago.
Patch to the bug. Removes the <label> tag.

Download all attachments as: .zip

Change History (4)

Patch to the bug. Removes the <label> tag.

  • Keywords has-patch added; label super admin user edit network removed
  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.5
  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [21927]:

Remove unnecessary label. Props waclawjacek. fixes #17978

Note: See TracTickets for help on using tickets.