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: unify BrowserWindow.isVisible()
logic cross-platform
#38242
Conversation
6fb7e1d
to
5278e2c
Compare
5278e2c
to
7458278
Compare
7458278
to
4e0c4e9
Compare
4e0c4e9
to
c7e4d33
Compare
macOS is failing on #38312 and this PR changes only Windows - merging. |
Release Notes Persisted
|
I have automatically backported this PR to "24-x-y", please check out #38313 |
I have automatically backported this PR to "25-x-y", please check out #38314 |
I have automatically backported this PR to "23-x-y", please check out #38315 |
Description of Change
Some debugging recently alerted me to the fact that
BrowserWindow.isVisible()
incorrectly differs cross-platform. On macOS, a window only returnstrue
forisVisible()
if the window is in the foreground and not minimized. However, on Windows, a window could be minimized and yet incorrectly show as visible. This corrects that schism.Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where
BrowserWindow.isVisible()
would incorrectly returntrue
for minimized windows on Windows.