Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@blasten
Copy link

@blasten blasten commented Jun 13, 2022

Ensures that accessibility events aren't forwarded to the parent view when
the embedded view should not be read by TalkBack.

Code sample
ExcludeSemantics(
    child: GoogleMap(
        ...
    ),
);

Fixes flutter/flutter#105658
Fixes flutter/flutter#29717

embeddedView.setLayoutParams(new FrameLayout.LayoutParams(physicalWidth, physicalHeight));
embeddedView.setLayoutDirection(request.direction);

// Accessibility is initially disabled, and it's e-enabled by AccessibilityBridge after
Copy link
Contributor

@GaryQian GaryQian Jun 14, 2022

Choose a reason for hiding this comment

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

This comment describes the scenario but does not clearly state why this must be set to disabled here specifically at this time. What is the value if it is not explicitly set here?

Copy link
Author

Choose a reason for hiding this comment

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

done

@blasten blasten added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Jun 14, 2022
@fluttergithubbot fluttergithubbot merged commit 888598e into flutter:main Jun 14, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

platform-android waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

3 participants