-
Notifications
You must be signed in to change notification settings - Fork 15k
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: ensure child window transparency works #28054
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome! 🙇♀️
Windows opened via window.open and intecepted via setWindowOpenHandler or the `new-window` event should (a) have the correct background color and (b) that background color should be transparent if specified. The changes in api_web_contents fix (a) and the changes in web_contents_preferences fix (b). Notes: Child windows with specified background colors or transpency now work as intended
6210403
to
c48e1e8
Compare
ab7b458
to
942974a
Compare
patches/chromium/moves_background_color_setter_of_webview_to_blinks_webprefs_logic.patch
Outdated
Show resolved
Hide resolved
patches/chromium/moves_background_color_setter_of_webview_to_blinks_webprefs_logic.patch
Outdated
Show resolved
Hide resolved
Should this also target 13-x-y? |
@VerteDinde has manually backported this PR to "12-x-y", please check out #28107 |
Release Notes Persisted
|
I have automatically backported this PR to "13-x-y", please check out #28112 |
Any update on 11-x-y? The problem is quite annoying but I don't have the expertise to backport it myself. |
/trop run backport-to 11-x-y |
The backport process for this PR has been manually initiated - sending your PR to |
I was unable to backport this PR to "11-x-y" cleanly; |
Removing needs-manual-bp/11-x-y since 11.x is security fixes only now. |
Windows opened via window.open and intecepted via setWindowOpenHandler or the
new-window
event should (a) have the correct background color and (b) that background color should be transparent if specified.The changes in api_web_contents fix (a) and the changes in web_contents_preferences fix (b).
Notes: Child windows with specified background colors or transpency now work as intended