Apply translation to accessibility tree when in landscape #5950
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jonahwilliams
changed the title
jonahwilliams
changed the title
jonahwilliams
requested review from
Hixie and
jason-simmons
and removed request for
jason-simmons
jason-simmons
approved these changes
Aug 7, 2018
liyuqian
added a commit
that referenced
this pull request
Aug 7, 2018
)" This reverts commit c7ce6dd.
liyuqian
added a commit
to liyuqian/flutter
that referenced
this pull request
Aug 7, 2018
7f083e5 Don't implicitly fall through in switch statement (flutter/engine#5964) c7ce6dd Apply translation to accessibility tree when in landscape (flutter/engine#5950) aef94b7 Reland "Updated background execution implementation for Android" (flutter/engine#5954) 3421bca Roll src/third_party/skia 36216fb0acbc..177742435e52 (15 commits) (flutter/engine#5963) 9d1e673 Temporarily add travis/analyze.sh back for Chrome bot (flutter/engine#5961) 4386afd Roll src/third_party/skia e6d0618f677c..36216fb0acbc (11 commits) (flutter/engine#5960) 7cef3da Roll src/third_party/skia 01d9a344b575..e6d0618f677c (1 commits) (flutter/engine#5958) ad82cb1 Roll src/third_party/skia fdf05f4ff4e9..01d9a344b575 (1 commits) (flutter/engine#5957) 6ea410c Revert "Roll Dart to 17b54c76ce9b945c6f013ad08c19268409c0694a (flutter/engine#5955)" (flutter#5956) f20c58f Roll Dart to 17b54c76ce9b945c6f013ad08c19268409c0694a (flutter/engine#5955) ed1938e Roll src/third_party/skia 641ac7daa81c..fdf05f4ff4e9 (3 commits) (flutter/engine#5953) 5f04e00 Remove travis directory (flutter/engine#5935) 5442c0a Revert "Updated background execution implementation for Android" (flutter/engine#5949) a5215ce Add hasImplicitScrolling SemanticFlag and support in Android bridge (flutter/engine#5941) 4681351 Roll src/third_party/skia 6e487e67a3f3..641ac7daa81c (16 commits) (flutter/engine#5948) bc885f3 Updated background execution implementation for Android 5770cbd Roll src/third_party/skia 12fb9cfeee07..6e487e67a3f3 (8 commits) (flutter/engine#5946) f4464a8 Roll src/third_party/skia 2e77f54f46e8..12fb9cfeee07 (1 commits) (flutter/engine#5945) c6baaaf Replace acquire+release thread annotation with excludes (flutter/engine#5944) 63ede2e Roll src/third_party/skia 59aabbcf3b0d..2e77f54f46e8 (1 commits) (flutter/engine#5943) 97aea09 Roll src/third_party/skia 71fe8baccc01..59aabbcf3b0d (1 commits) (flutter/engine#5942) aaf4a9a Roll src/third_party/skia 4c3b63e289c5..71fe8baccc01 (3 commits) (flutter/engine#5940)
liyuqian
added a commit
to flutter/flutter
that referenced
this pull request
Aug 7, 2018
* Roll engine to 7f083e5 7f083e5 Don't implicitly fall through in switch statement (flutter/engine#5964) c7ce6dd Apply translation to accessibility tree when in landscape (flutter/engine#5950) aef94b7 Reland "Updated background execution implementation for Android" (flutter/engine#5954) 3421bca Roll src/third_party/skia 36216fb0acbc..177742435e52 (15 commits) (flutter/engine#5963) 9d1e673 Temporarily add travis/analyze.sh back for Chrome bot (flutter/engine#5961) 4386afd Roll src/third_party/skia e6d0618f677c..36216fb0acbc (11 commits) (flutter/engine#5960) 7cef3da Roll src/third_party/skia 01d9a344b575..e6d0618f677c (1 commits) (flutter/engine#5958) ad82cb1 Roll src/third_party/skia fdf05f4ff4e9..01d9a344b575 (1 commits) (flutter/engine#5957) 6ea410c Revert "Roll Dart to 17b54c76ce9b945c6f013ad08c19268409c0694a (flutter/engine#5955)" (#5956) f20c58f Roll Dart to 17b54c76ce9b945c6f013ad08c19268409c0694a (flutter/engine#5955) ed1938e Roll src/third_party/skia 641ac7daa81c..fdf05f4ff4e9 (3 commits) (flutter/engine#5953) 5f04e00 Remove travis directory (flutter/engine#5935) 5442c0a Revert "Updated background execution implementation for Android" (flutter/engine#5949) a5215ce Add hasImplicitScrolling SemanticFlag and support in Android bridge (flutter/engine#5941) 4681351 Roll src/third_party/skia 6e487e67a3f3..641ac7daa81c (16 commits) (flutter/engine#5948) bc885f3 Updated background execution implementation for Android 5770cbd Roll src/third_party/skia 12fb9cfeee07..6e487e67a3f3 (8 commits) (flutter/engine#5946) f4464a8 Roll src/third_party/skia 2e77f54f46e8..12fb9cfeee07 (1 commits) (flutter/engine#5945) c6baaaf Replace acquire+release thread annotation with excludes (flutter/engine#5944) 63ede2e Roll src/third_party/skia 59aabbcf3b0d..2e77f54f46e8 (1 commits) (flutter/engine#5943) 97aea09 Roll src/third_party/skia 71fe8baccc01..59aabbcf3b0d (1 commits) (flutter/engine#5942) aaf4a9a Roll src/third_party/skia 4c3b63e289c5..71fe8baccc01 (3 commits) (flutter/engine#5940) * Sync framework changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Apply a translation to the accessibility tree based on the current decorView. When in reverse landscape on newer android devices, the system nav bar sticks to the left side of the screen. The accessibility system does not handle this automatically and we must apply the translation ourselves.
Still fully doesn't work for a landscape to landscape reverse transition - android does not consider this a configuration or size change so we may not get a chance to react. I mark the global geometry as dirty so that the next time there are any a11y updates the tree is aligned properly, but it may be temporarily misplaced.
Still strictly less broken than it was previously
Fixes flutter/flutter#19967