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: macOS tray icon alternate images #33026
Conversation
patches/chromium/fix_don_t_restore_maximized_windows_when_calling_showinactive.patch
Outdated
Show resolved
Hide resolved
4749362
to
a68ec28
Compare
a68ec28
to
9f14c40
Compare
hi, haven't merged into the main branch yet? |
9f14c40
to
2e573a4
Compare
Release Notes Persisted
|
I have automatically backported this PR to "15-x-y", please check out #33104 |
I have automatically backported this PR to "17-x-y", please check out #33105 |
I have automatically backported this PR to "16-x-y", please check out #33106 |
I have automatically backported this PR to "18-x-y", please check out #33107 |
Description of Change
Closes #33008.
Fixes an issue where
alternateImage
s did not work properly on macOS. To fix this, we need to change the button type when setting an alternate image because the default button type forNSStatusItem
,NSStatusBarButton
, does not display alternate content when clicked.NSButtonTypeMomentaryChange
displays its alternate content when clicked and returns to its normal content when the user releases it, which is the behavior users would expect when clicking a button with an alternate image set. After this fix, the tray icon displays the same behavior as it did prior to this breaking. Also tested with context menus etc to ensure nothing else broke by changing this.Checklist
npm test
passesRelease Notes
Notes: Fixes an issue where
alternateImage
s did not work properly on macOS.