fix(modal): focus the dialog wrapper on present so screen readers can enter the modal#31260

Open

gnbm wants to merge 22 commits into

mainfrom

FW-7611_investigation

Commits on Jul 1, 2026

fix(overlays): focus the dialog wrapper, not the role-less host, on present
gnbm
committed
fix(overlays): only redirect present() focus to a wrapper that is actually focusable
gnbm
committed
fix(popover): add default role="dialog" so aria-modal is not a no-op
gnbm
committed
fix(modal): scope dialog-wrapper focus to default modals; keep host focus for sheet/card
gnbm
committed
revert(popover): drop default role="dialog"; keep PR scoped to modal fix
gnbm
committed
test(modal): use idiomatic toBeFocused for the a11y focus assertion
gnbm
committed

Commits on Jul 2, 2026

Update comment in modal
gnbm
authored

Commits on Jul 7, 2026

Commits on Jul 8, 2026

Commits on Jul 13, 2026