Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Two things:
- There should be an onload event to set the focus to the username field.
- There needs to be tabindexes on both the username and password fields since right now when you tab from the username field, you end up on the Drupal affiliates link.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | drupal-20446-8.patch | 527 bytes | dead_arm |
Comments
Comment #1
cosmicdreams CreditAttribution: cosmicdreams commentedI agree with #1 but can't reproduce #2. I think we should alter our default themes to incorporate this change.
Comment #2
elv CreditAttribution: elv commentedIsn't this a duplicate of http://drupal.org/node/29060 ?
Comment #3
cosmicdreams CreditAttribution: cosmicdreams commented#2 is a duplicate of that issue and has been fixed. But #1 is still happening. My default drupal beta-2 site doesn't put the cursor on the login textbox when I go to it.
Perhaps we should triage this to the Drupal project?
Comment #4
StevenPatzFeatures go in 7.x
Comment #5
cosmicdreams CreditAttribution: cosmicdreams commentedThank you, I'll make note of that.
Comment #6
Jody LynnSetting the cursor focus on the login page sounds like a decent idea to me.
Comment #7
Tor Arne Thune CreditAttribution: Tor Arne Thune commentedStill a valid issue. I don't like having to tab or click every time I go to login and can imagine others feel the same way.
Comment #8
dead_arm#1174936: Allow FAPI usage of the autofocus attribute was won't fixed, so this would need to be added via #attributes.
Autofocus is HTML5 only, but that's not a problem for D8.
Comment #10
tim.plunkett#8: drupal-20446-8.patch queued for re-testing.
Comment #11
Jody LynnI approve this patch to this 7yo usability issue. Thank you dead_arm for joining core office hours!
Comment #12
catchNice! Committed/pushed to 8.x.
Comment #13
ZenDoodles CreditAttribution: ZenDoodles commentedDiscussion in IRC webchick said, "The biggest reason not to backport that is, I imagine it breaks XHTML validation." So we won't backport to D7.
Comment #16
mgiffordI was looking at autocomplete for #2096347: Add "autofocus" attribute to the module search and we're not being consistent about how we use autocomplete in Drupal 8 and we have no means to identify when it is appropriate to do so.
Right now it's used in most user forms, but not /user/register
Users should be able to predict when user forms will be presented. They need to have sufficient context such that screen reader users aren't confused by them.
Comment #17
mgiffordComment #18
mgiffordComment #20
AaronChristian CreditAttribution: AaronChristian at Acro Commerce commentedAnyone know the status of this task?
Comment #21
dawehnerWell it works for
/user/loginbut doesn't for/user/registerMoving to the right issue queue.
Comment #28
andrewmacpherson CreditAttribution: andrewmacpherson as a volunteer commentedtagging, just gathering issues which requested autofocus
Comment #29
andrewmacpherson CreditAttribution: andrewmacpherson as a volunteer commentedtCurrent status:
user/login - has autofocus, works
user/password - has autofocus, works
user/register - does not have autofocus.
We have #2861374: Inconsistent user interface with focusing fields to discuss this inconsistency, and it has more detail. So I'm closing his as a duplicate.
Comment #30
andrewmacpherson CreditAttribution: andrewmacpherson as a volunteer commented