Opened 10 years ago
Closed 10 years ago
#33647 closed defect (bug) (fixed)
On Add new user page, password text field does not get focussed on clicking 'New password' label
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | 4.3 |
Component: | Users | Keywords: | has-patch |
Focuses: | ui, accessibility, administration | Cc: |
Description
On Add new user page, Password has label and cursor pointer both. And there should not be cursor and label on password text.
There is no any effect after clicking on "Password" text. It is creating confusion to user after click.
Attachments (4)
Change History (21)
#1
@
10 years ago
- Component changed from General to Users
- Milestone changed from Awaiting Review to 4.4
- Version changed from trunk to 4.3
#2
@
10 years ago
This ticket look sounds good for user accessibility.
On user-edit.php page, New Password Page should not have label.
Added patch file for the same.
#4
@
10 years ago
By the way, when the field is displayed and then you click on the "hide" button to hide the password, the label will be correctly associated with the "pass1" field. When hidden instead, maybe the label should be associated with the "pass1-text" field. This probably got a bit messed up after the fix for IE 8 that split the field in two different fields (one with type "password" and one with type "text").
#5
@
10 years ago
Addind id
attribute to "Show password" button and changing <label for="that_id">
makes clicking a label act like the button itself - show the password. Imo, that's expected behaviour. It works fine when password is shown, however, when password is hidden it does not put the focus into the field. And that's strange.
#7
@
10 years ago
Totally agree with afercia ,referring https://core.trac.wordpress.org/ticket/33647#comment:4.
Label should be associated with "pass1-text" field also. Created the patch accordingly.
#8
follow-up:
↓ 12
@
10 years ago
I've reported same issue https://core.trac.wordpress.org/ticket/33778, but this issue is exists on all 'Generate password' button in 'profile.php', 'user-new.php' and also when we installing new wordpress 'install.php'. I've added patch for all pages in this ticket.
This ticket was mentioned in Slack in #core by juhisaxena. View the logs.
10 years ago
#10
@
10 years ago
- Summary changed from On Add new user page, Password has label and cursor pointer. to On Add new user page, password text field does not get focussed on clicking 'New password' label
#12
in reply to:
↑ 8
@
10 years ago
Replying to umesh.nevase:
I've reported same issue https://core.trac.wordpress.org/ticket/33778, but this issue is exists on all 'Generate password' button in 'profile.php', 'user-new.php' and also when we installing new wordpress 'install.php'. I've added patch for all pages in this ticket.
Not sure why you have created a new ticket ( https://core.trac.wordpress.org/ticket/33850 ) for the same issue even when it is addressed here. I have marked the other ticket as duplicate.
#14
in reply to:
↑ 13
;
follow-ups:
↓ 15
↓ 16
@
10 years ago
Replying to chiragswadia:
#33850 was marked as a duplicate.
In https://core.trac.wordpress.org/ticket/33778 I haven't got any reply for update profile page, so I created a new ticket https://core.trac.wordpress.org/ticket/33850.
#15
in reply to:
↑ 14
@
10 years ago
Replying to umesh.nevase:
Replying to chiragswadia:
#33850 was marked as a duplicate.
In https://core.trac.wordpress.org/ticket/33778 I haven't got any reply for update profile page, so I created a new ticket https://core.trac.wordpress.org/ticket/33850.
Since this is not a high priority blocking issue, it might take some time for any developer to give feedback. Opening a new ticket will not do any good. It will be better if you followup on the original ticket instead https://core.trac.wordpress.org/ticket/33778
#16
in reply to:
↑ 14
@
10 years ago
Replying to chiragswadia:
Replying to umesh.nevase:
Replying to chiragswadia:
#33850 was marked as a duplicate.
In https://core.trac.wordpress.org/ticket/33778 I haven't got any reply for update profile page, so I created a new ticket https://core.trac.wordpress.org/ticket/33850.
Since this is not a high priority blocking issue, it might take some time for any developer to give feedback. Opening a new ticket will not do any good. It will be better if you followup on the original ticket instead https://core.trac.wordpress.org/ticket/33778
@umesh.nevase and @chiragswadia #33778 was part of the 4.3.1 milestone that shipped ~22 hours ago, please do not reopen this ticket, if needed please reopen #33850 if it is not a duplicate of this ticket.
patch file after removed label