github/electron
1
1
Fork 0
mirror of https://github.com/electron/electron.git synced 2026-05-02 03:00:22 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix: preserve transparency across setResizable toggles on Windows (#51217)

Browse Source 
After #49428 made `NativeWindowViews::CanResize()` return `resizable_`
for frameless windows (instead of `resizable_ && thick_frame_`),
`HWNDMessageHandler::SizeConstraintsChanged()` started adding
`WS_THICKFRAME` to the window style whenever `CanResize()` reported true.
`WS_THICKFRAME` is incompatible with layered (translucent) windows and
destroys their transparency.

`SetContentSizeConstraints` already guards against this by skipping
`OnSizeConstraintsChanged()` when `!thick_frame_`. `SetResizable` did
not, so toggling resizability on a transparent window (e.g.
`setResizable(false)` then `setResizable(true)`) caused the Chromium
path to add `WS_THICKFRAME` and strip transparency.

Apply the same guard in `SetResizable`. Min/max constraints are still
enforced — Chromium reads them from the widget delegate on every
`WM_GETMINMAXINFO`, independent of `SizeConstraintsChanged()`.

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
This commit is contained in:
trop[bot]
2026-04-22 12:33:06 +02:00
committed by GitHub
parent 35aea3afcc
commit 2c05608a76
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
shell/browser/native_window_views.cc
View File

@@ -965,10 +965,18 @@ void NativeWindowViews::SetResizable(bool resizable) {
gfx::Size window_size = GetSize();
SetSizeConstraints(extensions::SizeConstraints(window_size, window_size));
}
// Forcing OnSizeConstraintsChanged on Windows when !thick_frame_ would
// cause HWNDMessageHandler::SizeConstraintsChanged() to add WS_THICKFRAME
// based on CanResize(), which destroys transparency on layered windows.
// Min/max are still enforced through WM_GETMINMAXINFO directly from the
// widget delegate.
#if BUILDFLAG(IS_WIN)
if (thick_frame_ && widget() && widget()->widget_delegate())
widget()->OnSizeConstraintsChanged();
UpdateThickFrame();
#else
if (widget() && widget()->widget_delegate())
widget()->OnSizeConstraintsChanged();
#if BUILDFLAG(IS_WIN)
UpdateThickFrame();
#endif
}
}