Make WordPress Core

Opened 15 years ago

Closed 14 years 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: Focuses: multisite

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 15 years ago.
Patch to the bug. Removes the <label> tag.

Download all attachments as: .zip

Change History (4)

@waclawjacek
15 years ago

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

#1 @kawauso
15 years ago

  • Keywords has-patch added; label super admin user edit network removed

#2 @nacin
14 years ago

  • Keywords commit added
  • Milestone Awaiting Review3.5

#3 @ryan
14 years ago

  • Owner set to ryan
  • Resolutionfixed
  • Status newclosed

In [21927]:

Remove unnecessary label. Props waclawjacek. fixes #17978

Note: See TracTickets for help on using tickets.