Skip to content

Conversation

@swift-kim
Copy link
Member

Cherry-pick flutter#32332.

@swift-kim swift-kim changed the title Backport "Sync accessibility features of window.dart and embedder.h" Backport "Sync accessibility flags in embedder.h" Apr 7, 2022
@swift-kim swift-kim force-pushed the backport-high-contrast branch 2 times, most recently from 26aa344 to 89b46e7 Compare April 7, 2022 01:37
@bbrto21
Copy link

bbrto21 commented Apr 7, 2022

Is this CI failure on Azure same as #266 ?

@swift-kim
Copy link
Member Author

@bbrto21 Yes, #267 should be merged first. Could you approve #267?

The following enum types in `embedder.h` must match with the corresponding Dart/C++ classes in `dart:ui` but some values are missing.

- `FlutterAccessibilityFeature`
- `FlutterSemanticsAction`
- `FlutterSemanticsFlag`

The comments say

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/shell/platform/embedder/embedder.h#L83

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/lib/ui/window.dart#L784-L785
@swift-kim swift-kim force-pushed the backport-high-contrast branch from 89b46e7 to 2addb74 Compare April 7, 2022 10:27
@bbrto21 bbrto21 merged commit 3038c66 into flutter-tizen:flutter-2.10.1-tizen Apr 7, 2022
swift-kim added a commit that referenced this pull request May 12, 2022
The following enum types in `embedder.h` must match with the corresponding Dart/C++ classes in `dart:ui` but some values are missing.

- `FlutterAccessibilityFeature`
- `FlutterSemanticsAction`
- `FlutterSemanticsFlag`

The comments say

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/shell/platform/embedder/embedder.h#L83

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/lib/ui/window.dart#L784-L785
swift-kim added a commit that referenced this pull request Aug 5, 2022
The following enum types in `embedder.h` must match with the corresponding Dart/C++ classes in `dart:ui` but some values are missing.

- `FlutterAccessibilityFeature`
- `FlutterSemanticsAction`
- `FlutterSemanticsFlag`

The comments say

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/shell/platform/embedder/embedder.h#L83

https://github.com/flutter/engine/blob/2d29e2f3b5e4c951809eb7578ff5f114f91efbeb/lib/ui/window.dart#L784-L785
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