Skip to content

fix: all active element access for shadow dom#9608

Merged
snowystinger merged 8 commits intomainfrom
fix-all-document-active-element
Feb 9, 2026
Merged

fix: all active element access for shadow dom#9608
snowystinger merged 8 commits intomainfrom
fix-all-document-active-element

Conversation

@snowystinger
Copy link
Member

Closes

As part of the series #9551 and #9485

The three of them combined should close out Shadow dom event handling

changes all document.activeElement to getActiveElement() which will default to using document.

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@rspbot
Copy link

rspbot commented Feb 5, 2026

reidbarber
reidbarber previously approved these changes Feb 9, 2026
# Conflicts:
#	packages/@react-aria/numberfield/src/useNumberField.ts
#	packages/@react-aria/utils/src/useViewportSize.ts
#	packages/@react-spectrum/menu/src/ContextualHelpTrigger.tsx
#	packages/@react-spectrum/menu/src/SubmenuTrigger.tsx
#	packages/@react-spectrum/menu/src/useOverlayPosition.ts
LFDanLu
LFDanLu previously approved these changes Feb 9, 2026
reidbarber
reidbarber previously approved these changes Feb 9, 2026
# Conflicts:
#	eslint.config.mjs
#	packages/@react-aria/calendar/src/useRangeCalendar.ts
#	packages/@react-aria/dialog/src/useDialog.ts
#	packages/@react-aria/grid/src/useGridCell.ts
#	packages/@react-aria/gridlist/src/useGridListItem.ts
#	packages/@react-aria/menu/src/useSubmenuTrigger.ts
#	packages/@react-aria/overlays/src/useOverlayPosition.ts
#	packages/@react-aria/selection/src/useSelectableCollection.ts
#	packages/@react-spectrum/menu/src/ContextualHelpTrigger.tsx
#	packages/@react-spectrum/menu/src/SubmenuTrigger.tsx
#	packages/@react-spectrum/s2/src/TableView.tsx
#	packages/@react-spectrum/table/src/TableViewBase.tsx
#	packages/dev/eslint-plugin-rsp-rules/index.js
#	packages/react-aria-components/src/Popover.tsx
@snowystinger snowystinger dismissed stale reviews from reidbarber and LFDanLu via bc47252 February 9, 2026 20:25
@rspbot
Copy link

rspbot commented Feb 9, 2026

@snowystinger snowystinger added this pull request to the merge queue Feb 9, 2026
Merged via the queue into main with commit b66f676 Feb 9, 2026
29 checks passed
@snowystinger snowystinger deleted the fix-all-document-active-element branch February 9, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants