mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

fix: window accentColor should adhere to native window behavior (#47802)

Browse source 
* fix: window accentColor should adhere to native window behavior

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

* fix: address review feedback

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

* chore: remove duplicate UpdateWindowAccentColor call in ctor

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

---------

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] 2025-07-21 12:37:01 +02:00 • committed by GitHub
commit 673ec5d39e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 11 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

8
shell/browser/native_window_views_win.cc
View file

@ -499,7 +499,7 @@ bool NativeWindowViews::PreHandleMSG(UINT message,
return false;
}
case WM_DWMCOLORIZATIONCOLORCHANGED: {
UpdateWindowAccentColor();
UpdateWindowAccentColor(IsActive());
return false;
}
case WM_SETTINGCHANGE: {
@ -507,7 +507,7 @@ bool NativeWindowViews::PreHandleMSG(UINT message,
const wchar_t* setting_name = reinterpret_cast<const wchar_t*>(l_param);
std::wstring setting_str(setting_name);
if (setting_str == L"ImmersiveColorSet")
UpdateWindowAccentColor();
UpdateWindowAccentColor(IsActive());
}
return false;
}
@ -570,7 +570,7 @@ void NativeWindowViews::HandleSizeEvent(WPARAM w_param, LPARAM l_param) {
}
}
void NativeWindowViews::UpdateWindowAccentColor() {
void NativeWindowViews::UpdateWindowAccentColor(bool active) {
if (base::win::GetVersion() < base::win::Version::WIN11)
return;
@ -589,7 +589,7 @@ void NativeWindowViews::UpdateWindowAccentColor() {
should_apply_accent = std::get<bool>(accent_color_);
} else if (std::holds_alternative<std::monostate>(accent_color_)) {
// If no explicit color was set, default to the system accent color.
should_apply_accent = IsAccentColorOnTitleBarsEnabled();
should_apply_accent = IsAccentColorOnTitleBarsEnabled() && active;
}
// Use system accent color as fallback if no explicit color was set.