![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#62906 closed enhancement (fixed)
Allow :focus-visible pseudo-selector to be set in theme.json
| Reported by: | huubl | Owned by: | joedolson |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.8 |
| Component: | Themes | Version: | |
| Severity: | normal | Keywords: | has-patch commit gutenberg-merge |
| Cc: | Focuses: | accessibility, css |
Description (last modified by )
See also the paired Gutenberg PR: https://github.com/WordPress/gutenberg/pull/68521
These should be handled together.
Change History (19)
This ticket was mentioned in PR #8261 on WordPress/wordpress-develop by @huubl.
17 months ago
#1
- Keywords has-patch added
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
17 months ago
@audrasjb commented on PR #8261:
17 months ago
#3
Related Gutenberg PR: https://github.com/WordPress/gutenberg/pull/68521
#6
@
17 months ago
- Component General → Themes
- Focuses css added
- Keywords commit added
As per today's accessibility bug scrub, I added the related PR to milestone 6.8. We can commit this ticket and merge the related PR in whatever order we prefer.
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
17 months ago
@joedolson commented on PR #8261:
17 months ago
#9
in r59854
17 months ago
#11
@joedolson I think my last commit is missing here.
#12
@
16 months ago
- Keywords has-patch, commit, gutenberg-merge → has-patch commit gutenberg-merge
- Resolution fixed
- Status closed → reopened
@joedolson @joemcgill @audrasjb the changes r59854 are incomplete / not including the changes as merged in https://github.com/WordPress/gutenberg/pull/68521. The last commit is missing here. Can you please take a look?
#13
@
16 months ago
Thanks for the heads up, @huubl. @joedolson do you have time to take another look? If not, feel free to reassign to me.
#14
@
16 months ago
@joemcgill Yeah, I can take care of that. That's a weird thing to have omitted, but I don't recall having seen that change at all...
#15
@
16 months ago
Oh, I see - that's the Gutenberg merge PR, not the one I merged. Checking on it.
#16
@
16 months ago
OK; I see that PR 8261 never got updated with later changes from the upstream PR, and I overlooked that. Straightforward; just needs to be checked and committed.
This ticket was mentioned in PR #8548 on WordPress/wordpress-develop by @joedolson.
16 months ago
#17
Trac ticket: https://core.trac.wordpress.org/ticket/62906
Trac ticket: https://core.trac.wordpress.org/ticket/62906
Gutenberg ticket: https://github.com/WordPress/gutenberg/pull/68521