Skip to content

fix: expose iOS footer accessibility elements#700

Merged
lodev09 merged 3 commits into
lodev09:mainfrom
erickreutz:eric/expose-footer-accessibility-elements
May 30, 2026
Merged

fix: expose iOS footer accessibility elements#700
lodev09 merged 3 commits into
lodev09:mainfrom
erickreutz:eric/expose-footer-accessibility-elements

Conversation

@erickreutz
Copy link
Copy Markdown
Contributor

@erickreutz erickreutz commented May 30, 2026

Summary

Fixes iOS footer accessibility traversal for nested footer controls by exposing footer child accessibility elements through the sheet container, recalculating the sheet accessibility container after footer layout changes, and restoring the previous window accessibility elements when the sheet dismisses.

This builds on #699 for cases where footer controls need to remain reachable by XCTest and assistive technologies while the sheet is presented.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Test Plan

Ran PATH="$(dirname "$(xcrun --find clang-format)"):$PATH" scripts/objclint.sh.
Ran git diff --check.
Validated the equivalent patch in Lugg iOS app E2E coverage.

Screenshots / Videos

N/A

Checklist

  • I tested on iOS
  • I tested on Android
  • I tested on Web
  • I updated the documentation (if needed)
  • I added a changelog entry (if needed)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

@erickreutz is attempting to deploy a commit to the lodev09 Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-native-true-sheet Ready Ready Preview, Comment May 30, 2026 2:19pm

Request Review

@lodev09 lodev09 merged commit 0d9f4e4 into lodev09:main May 30, 2026
6 checks passed
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