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
fix: persist BrowserView background color when bounds offscreen #30510
Conversation
It's great, very grateful👍 👍 👍 👍 👍 👍 |
failure is a flake |
Release Notes Persisted
|
I have automatically backported this PR to "13-x-y", please check out #30540 |
I have automatically backported this PR to "14-x-y", please check out #30541 |
I have automatically backported this PR to "15-x-y", please check out #30542 |
Reverting change introduced in PR: electron#30510
Reverting change introduced in PR: #30510 Co-authored-by: Andreas Johansson <aj3621@tobii.com>
Reverting change introduced in PR: #30510
Reverting change introduced in PR: #30510
Reverting change introduced in PR: #30510 Co-authored-by: Andreas Johansson <aj3621@tobii.com>
Reverting change introduced in PR: #30510 Co-authored-by: Andreas Johansson <aj3621@tobii.com>
…tron#32747) Reverting change introduced in PR: electron#30510 Co-authored-by: Andreas Johansson <aj3621@tobii.com>
…tron#32747) Reverting change introduced in PR: electron#30510 Co-authored-by: Andreas Johansson <aj3621@tobii.com>
Description of Change
Closes #29778.
Fixes an issue where background color would not be correctly applied to
BrowserViews
on Windows when either thex
ory
coordinate is negative (off-screen).This was happening because
view::Views
was trying to paint the whole content bounds instead of the visible bounds, which caused issues because when part of the bounds are offscreen the compositor overrides and paints them white. This fixes that by painting the visible bounds.Before:
After:
Tested with https://gist.github.com/712a2fde2757736c9dbf6c3e7e0aa18b.
Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where background color would not be correctly applied to
BrowserViews
on Windows when either thex
ory
coordinate is negative (off-screen).