Skip to content

[BottomSheetController] Explicitly set isAccessibilityElement of resizingHandleView#2227

Merged
nodes11 merged 2 commits into
microsoft:mainfrom
nodes11:fix_bottom_sheet_handle_a11y
Oct 27, 2025
Merged

[BottomSheetController] Explicitly set isAccessibilityElement of resizingHandleView#2227
nodes11 merged 2 commits into
microsoft:mainfrom
nodes11:fix_bottom_sheet_handle_a11y

Conversation

@nodes11

@nodes11 nodes11 commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Platforms Impacted

  • iOS
  • visionOS
  • macOS

Description of changes

When the isExpandable state of the BottomSheetController changes to false, the resizingHandleView hides but its frame remains an accessible element in VoiceOver mode.

This change sets the isAccessibilityElement property when isExpandable changes.

Binary change

N/A

Verification

Validated in Fluent Demo Controller that the issue no longer reproduces.

Visual Verification
Before After

|

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2025-10-27.at.10.04.50.mov

|

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2025-10-27.at.10.03.35.mov

|

Pull request checklist

This PR has considered:

  • Light and Dark appearances
  • iOS supported versions (all major versions greater than or equal current target deployment version)
  • VoiceOver and Keyboard Accessibility
  • Internationalization and Right to Left layouts
  • Different resolutions (1x, 2x, 3x)
  • Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
  • iPad Pointer interaction
  • SwiftUI consumption (validation or new demo scenarios needed)
  • Objective-C exposure (provide it only if needed)
Microsoft Reviewers: Open in CodeFlow

@nodes11 nodes11 requested a review from a team as a code owner October 27, 2025 17:10
@nodes11 nodes11 enabled auto-merge (squash) October 27, 2025 19:02
@nodes11 nodes11 merged commit cc00c74 into microsoft:main Oct 27, 2025
7 checks passed
@joannaquu joannaquu mentioned this pull request Jul 9, 2026
12 tasks
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