-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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: restore non-panel focus behavior #42180
Conversation
Fixes #42157 |
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.
I concur, the remaining IsPanel()
safety checks should be enough (plus, we have a test for this, so if that passes, we know we'll be okay)
Release Notes Persisted
|
I have automatically backported this PR to "31-x-y", please check out #42186 |
I have automatically backported this PR to "30-x-y", please check out #42187 |
Thank you for the quick fix! 🙏 |
The new activate API on macOS is pretty bad, we should just keep using the old API. Similar to #42180. Restores non-panel behavior to pre-panel support. Notes: BrowserWindow.show() now correctly restores focus to inactive apps on macOS
The new activate API on macOS is pretty bad, we should just keep using the old API. Similar to #42180. Restores non-panel behavior to pre-panel support. Notes: BrowserWindow.show() now correctly restores focus to inactive apps on macOS Co-authored-by: Samuel Attard <marshallofsound@electronjs.org>
The new activate API on macOS is pretty bad, we should just keep using the old API. Similar to #42180. Restores non-panel behavior to pre-panel support. Notes: BrowserWindow.show() now correctly restores focus to inactive apps on macOS Co-authored-by: Samuel Attard <marshallofsound@electronjs.org>
The new activate API on macOS is pretty bad, we should just keep using the old API. Similar to #42180. Restores non-panel behavior to pre-panel support. Notes: BrowserWindow.show() now correctly restores focus to inactive apps on macOS Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Samuel Attard <marshallofsound@electronjs.org>
The new activate API on macOS is pretty bad, we should just keep using the old API. Similar to #42180. Restores non-panel behavior to pre-panel support. Notes: BrowserWindow.show() now correctly restores focus to inactive apps on macOS Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Samuel Attard <marshallofsound@electronjs.org>
Fixes an issue caused by the chain of #40307 and #41750
Fixes #42157
With the new "only run this on non-panels" logic we can restore the exact same old codepath.
Notes:
BrowserWindow.focus()
now correctly restore focus to inactive apps on macOS