Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved support for VoiceOver/TalkBack #417

Merged
merged 1 commit into from Sep 2, 2022

Conversation

JhonyPerez
Copy link
Contributor

Description:

  • When wrapped in the Positioned widget, the suggestions box widget is placed before the Scaffold semantically. In order to have the suggestions box navigable from the search input or keyboard, Semantics > Align > ConstrainedBox are needed. This does not change the style visually. However, when VO/TB are not enabled it is necessary to use the Positioned widget to allow the elements to be tappable.

Before:

Untitled.1.mp4

After:

RPReplay_Final1659357543.MP4

@sjmcdowall sjmcdowall merged commit 5ad93e4 into AbdulRahmanAlHamali:master Sep 2, 2022
// the style visually. However, when VO/TB are not enabled it is
// necessary to use the Positioned widget to allow the elements to be
// properly tappable.
return MediaQuery.of(context).accessibleNavigation
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause appium to be unable to select item

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? What exactly is the issue? To a human this appears to look decent and okay ...

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need more time to pinpoint the root cause, but for now it looks like Align is causing the problem.

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.

None yet

3 participants