Skip to content

Manually set accessibility focus after layout (not after a time delay) when jumping between level 1 comments - #1375

Merged
QuantumBadger merged 2 commits into
QuantumBadger:masterfrom
codeofdusk:fix-a11y-l1-actions
Sep 13, 2026
Merged

QuantumBadger merged 2 commits into
QuantumBadger:masterfrom
codeofdusk:fix-a11y-l1-actions

Conversation

@codeofdusk

@codeofdusk codeofdusk commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

This PR sets accessibility focus after the recycler finishes laying out when jumping between level 1 comments, removing the after-800-ms hack, and refactors the logic to make way for future work.

@codeofdusk codeofdusk closed this Aug 23, 2026
@codeofdusk codeofdusk reopened this Aug 23, 2026
@codeofdusk
codeofdusk force-pushed the fix-a11y-l1-actions branch from 56c034d to 6ed6a85 Compare August 23, 2026 05:45
@codeofdusk codeofdusk changed the title Make next/previous comment actions relative to the invoking comment and manually set accessibility focus Manually set accessibility focus after layout (not after a time delay) when jumping between level 1 comments Aug 23, 2026
@codeofdusk
codeofdusk marked this pull request as ready for review August 23, 2026 10:58
@codeofdusk

Copy link
Copy Markdown
Contributor Author

CC @QuantumBadger

@QuantumBadger
QuantumBadger merged commit 3bcce5a into QuantumBadger:master Sep 13, 2026
@QuantumBadger

Copy link
Copy Markdown
Owner

Thank you! Works well in my testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants