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: BrowserWindow backgroundColor #30778
fix: BrowserWindow backgroundColor #30778
Conversation
1b87f50
to
84e32ef
Compare
819dae6
to
cfb39c6
Compare
@nornagon I found a better approach for this. To make a WebContents transparent, we only need to set the background color to It also doesn't make sense to enforce transparency of the WebContents based on the options passed to BrowserWindow. The user should be able to change the background color to not be transparent at any time. With these changes, WebContents transparency via |
cfb39c6
to
78a93c1
Compare
No Release Notes |
Description of Change
fixes #30759
Port of #30777 which targets v14+15. I manually ported this to latest because of the changes to WebPreferences in #30193
The code below would fail to convert a hex string to an
SkColor
which resulted in the RVHW's background color always being set to transparent.electron/shell/browser/web_contents_preferences.cc
Line 227 in dd7aeda
cc @codebytere @nornagon
Checklist
npm test
passesRelease Notes
Notes: none