Opened 9 years ago
Closed 4 months ago
#34957 closed enhancement (fixed)
#a11y-focus: Standardizing the handling of :focus and :hover
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.5 | Priority: | normal |
Severity: | normal | Version: | 4.4 |
Component: | Administration | Keywords: | color-contrast |
Focuses: | ui, accessibility | Cc: |
Description
After the discussion of 34876 with @mor10 and @michaelarestad, we've realized there are two main cases:
- Times when the :focus and :hover states can be the same
- Times when they need to be different
In order to tackle the general :focus problem we'll need to:
- Define which elements fall into which category
- Establish a standard implementation for each category
- Implement the new standards
This could quickly evolve into a larger project, but we want to start resolve the :focus/:hover to make sure that gets addressed. This ticket is intended to start the conversation and put the attention on :focus.
This is an important a11y issue, as 34876 shows there are still elements that lack a :focus state completely. As part of this plan, we'll identify those elements lacking :focus and address both the lack of :focus and the correct implementation at the same time.
Change History (12)
This ticket was mentioned in Slack in #accessibility by adamsoucie. View the logs.
9 years ago
This ticket was mentioned in Slack in #accessibility by adamsoucie. View the logs.
9 years ago
#3
@
9 years ago
- Milestone changed from Awaiting Review to 4.5
- Owner set to adamsoucie
- Status changed from new to assigned
- Version changed from trunk to 4.4
This ticket was mentioned in Slack in #accessibility by rianrietveld. View the logs.
9 years ago
#6
@
9 years ago
Added a quick patch to #34904. Would love some feedback.
After that, I think we can go through each interactive element (starting with buttons, links, and form fields) and then move on to menus and other misc links to make sure the focus state change (background/text/border) is clear and consistent.
This ticket was mentioned in Slack in #core by jorbin. View the logs.
9 years ago
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
4 months ago
#12
@
4 months ago
- Milestone changed from Future Release to 6.5
- Resolution set to fixed
- Status changed from assigned to closed
Step 1 was a decision, Step 2 was completed in #34904, and Step 3 was completed in #51870. So I believe that this ticket is actually finished, and can be closed as completed.
There are undoubtedly still some focus states that are lingering; but those should be treated as individual tickets moving forward, rather than blockers to closing this overarching task.
Based on today's a11y Slack chat, we're establishing :focus as the standard, with :hover being somewhat of an extension of :focus.
To quote Joe Dolson:
Step 1 has been decided that :focus and :hover need to be different in all cases.
Moving on to Step 2, due to 34904. This provides us with the standard to implement. Once that's complete, we'll move to Step 3.