Skip to content

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Oct 16, 2024

This test relies on the enumeration of all engine flags, and it will fail when flutter/engine#55780 adds SemanticsFlag.hasSelectedState. So temporarily skip the test to allow the new flag to be added. I will later unskip the test and change it to check for the new flag as well.

A step towards #66673

@yjbanov yjbanov requested a review from hannah-hyj October 16, 2024 18:28
@github-actions github-actions bot added framework flutter/packages/flutter repository. See also f: labels. a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) labels Oct 16, 2024
Copy link
Member

@hannah-hyj hannah-hyj left a comment

Choose a reason for hiding this comment

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

LGTM!

@yjbanov yjbanov added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 16, 2024
@auto-submit auto-submit bot merged commit 4a57ce7 into flutter:master Oct 16, 2024
auto-submit bot pushed a commit that referenced this pull request Oct 16, 2024
Lurchfresser pushed a commit to Lurchfresser/flutter that referenced this pull request Oct 17, 2024
…tter#157017)

This test relies on the enumeration of all engine flags, and it will fail when flutter/engine#55780 adds `SemanticsFlag.hasSelectedState`. So temporarily skip the test to allow the new flag to be added. I will later unskip the test and change it to check for the new flag as well.

A step towards flutter#66673
Lurchfresser pushed a commit to Lurchfresser/flutter that referenced this pull request Oct 17, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 17, 2024
auto-submit bot pushed a commit to flutter/engine that referenced this pull request Oct 17, 2024
Introduce `SemanticsFlag.hasSelectedState` to the engine API. At this point the flag is not backed by any engine functionality. See also: flutter/flutter#66673 (comment)

| ⚠️ WARNING           |
|:----------------------------|
| Submit _AFTER_ flutter/flutter#157017 and flutter/flutter#157061 land in the framework     |

A step towards flutter/flutter#66673
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…ngine#55780)

Introduce `SemanticsFlag.hasSelectedState` to the engine API. At this point the flag is not backed by any engine functionality. See also: flutter#66673 (comment)

| ⚠️ WARNING           |
|:----------------------------|
| Submit _AFTER_ flutter#157017 and flutter#157061 land in the framework     |

A step towards flutter#66673
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants