#33059 closed defect (bug) (fixed)
Publish box timestamp form elements initial focus
Reported by: | afercia | Owned by: | iseulde |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Posts, Post Types | Keywords: | has-patch |
Focuses: | ui, javascript | Cc: |
Description
In the Post Publish box, when opening the timestamp expandable div, focus is moved to the "Month" select. This is correct when the language set is English.
Turns out translators can swap the fields to adapt them to their locale date format, for example see the Italian publish box in the screenshot below:
I guess Italian is not the only case and we shouldn't assume the Month select will always be the first form element.
Attachments (2)
Change History (11)
#2
@
9 years ago
Noticed that sometimes screen readers have issues when focus is moved to elements that are still not fully visible. Focus should always be moved when the animation has completed. Will open a new ticket for this.
#3
follow-up:
↓ 4
@
9 years ago
- Milestone changed from Awaiting Review to Future Release
- Version 4.2 deleted
Why input:visible
?
#4
in reply to:
↑ 3
@
9 years ago
Replying to iseulde:
Why
input:visible
?
There are some hidden inputs in the timestamp div, just wanted to be sure to exclude them. And yes they come after so :visible
maybe is unnecessary but who knows maybe some future markup change could place some hidden inputs before. On the other hand, targeting the inner .timestamp-wrap
would mean getting one more element. Oh well maybe I'm overthinking, feel free to change it and keep it simple if you think that would be better :)
Proposed patch moves focus to the first, visible, input or select element.