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

Preserve the alpha for VD content by setting a transparent background. #9849

Merged
merged 1 commit into from
Jul 16, 2019

Conversation

amirh
Copy link
Contributor

@amirh amirh commented Jul 16, 2019

The previous attempt to do this by setting the theme to
Theme_Translucent_NoTitleBar was reported to offsetting touch events on
specific Xiaomi devices.

This is an alternative approach to #9828 which is reported to offset touch events on some Xiaomi devices(#7236 (comment)).

The previous attempt to do this by setting the theme to
Theme_Translucent_NoTitleBar was reported to offsetting touch events on
specific Xiaomi devices.
@amirh amirh requested a review from mklim July 16, 2019 19:13
Copy link
Contributor

@mklim mklim left a comment

Choose a reason for hiding this comment

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

LGTM

@amirh amirh merged commit 7ae3c3e into flutter:master Jul 16, 2019
@amirh amirh deleted the translucent_xiaomi branch July 16, 2019 19:45
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 17, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jul 17, 2019
flutter/engine@66092d5...e3e4b26

git log 66092d5..e3e4b26 --no-merges --oneline
e3e4b26 Roll fuchsia/sdk/core/mac-amd64 from pwsYtqWGHRP43dB0EyLu7S9R3uGV0dIhulPbnSUDUmcC to xJPmd7maxxcPRu9HpByelwYaqa-v6qipCfy6JhgYS9oC (flutter/engine#9858)
3233f94 Disable Fuchsia Debug & Release presubmits and only attempt the Profile unopt variant. (flutter/engine#9856)
6e1a3e8 Roll src/third_party/skia a3e2996b0834..947efe28de74 (6 commits) (flutter/engine#9853)
9eed783 Selectively enable tests that work on Windows and file issues for ones that don't. (flutter/engine#9852)
7ae3c3e Preserve the alpha for VD content by setting a transparent background. (flutter/engine#9849)
a3926a3 Roll fuchsia/sdk/core/mac-amd64 from 94o0_NIcwi-V68SmUN4hU0fX5_JkMpPU5V8HaFvKWfYC to pwsYtqWGHRP43dB0EyLu7S9R3uGV0dIhulPbnSUDUmcC (flutter/engine#9848)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (aaclarke@google.com), and stop
the roller if necessary.
johnsonmh pushed a commit to johnsonmh/flutter that referenced this pull request Jul 30, 2019
flutter/engine@66092d5...e3e4b26

git log 66092d5..e3e4b26 --no-merges --oneline
e3e4b26 Roll fuchsia/sdk/core/mac-amd64 from pwsYtqWGHRP43dB0EyLu7S9R3uGV0dIhulPbnSUDUmcC to xJPmd7maxxcPRu9HpByelwYaqa-v6qipCfy6JhgYS9oC (flutter/engine#9858)
3233f94 Disable Fuchsia Debug & Release presubmits and only attempt the Profile unopt variant. (flutter/engine#9856)
6e1a3e8 Roll src/third_party/skia a3e2996b0834..947efe28de74 (6 commits) (flutter/engine#9853)
9eed783 Selectively enable tests that work on Windows and file issues for ones that don&flutter#39;t. (flutter/engine#9852)
7ae3c3e Preserve the alpha for VD content by setting a transparent background. (flutter/engine#9849)
a3926a3 Roll fuchsia/sdk/core/mac-amd64 from 94o0_NIcwi-V68SmUN4hU0fX5_JkMpPU5V8HaFvKWfYC to pwsYtqWGHRP43dB0EyLu7S9R3uGV0dIhulPbnSUDUmcC (flutter/engine#9848)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (aaclarke@google.com), and stop
the roller if necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants