Skip to content

Conversation

@janewman
Copy link
Contributor

@janewman janewman commented Oct 7, 2025

Major Changes in this PR:

  • Aligns document around the idea that each focusgroup segment will have a single, guaranteed tab stop, even if the element would normally be focusable via sequential tab navigation, e.g. natively focusable or a tabindex >= 0. This means:
    • Authors don't have to manage tabindex outside of ensuring their items are focusable and managing first-entry order.
    • Users can not tab between focusgroup items in the same segment, even if they have a positive tabindex and would otherwise expect to be in sequential tab order.
  • Renames this page to "Focusgroup" and removes the other version of the explainer from the sidebar (still linkable from this document), see: Should "Inactive" be more obvious? #1277 (comment).
  • Removes grid examples outside the grid section, and makes the contents of the grid section collapsed. [focusgroup] Should we remove the "future considerations" out of the new explainer? #1295
  • Misc. Adds missing or broken links, allows table of contents to be hidden, misc editorial.

@janewman janewman changed the title Further clarifications [focusgroup] General clarifications and edits, move previous explainer out of the sidebar Oct 7, 2025
@janewman janewman force-pushed the further_clarifications branch from bab95aa to 1a756b5 Compare October 7, 2025 18:48
@janewman janewman marked this pull request as ready for review October 7, 2025 19:07
Co-authored-by: Philipp Gfeller <[email protected]>
@janewman
Copy link
Contributor Author

@mfreed7 or @gregwhitworth (or any other owner) PTAL and let me know what changes are needed, or please hit "submit" if everything looks good.

@mfreed7 mfreed7 merged commit fdd348a into openui:main Oct 24, 2025
5 checks passed
@janewman janewman deleted the further_clarifications branch October 24, 2025 20:34
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.

3 participants