Skip to content
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 #38315

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented May 16, 2023

Backport of #38242

See that PR for details.

Notes: Fixed an issue where BrowserWindow.isVisible() would incorrectly return true for minimized windows on Windows.

@trop trop bot requested a review from codebytere May 16, 2023 09:47
@trop trop bot added 23-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels May 16, 2023
@codebytere codebytere force-pushed the trop/23-x-y-bp-fix-unify-browserwindow-isvisible-logic-cross-platform-1684230426706 branch 2 times, most recently from cd732a2 to 940a233 Compare May 16, 2023 12:57
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@jkleinsc jkleinsc force-pushed the trop/23-x-y-bp-fix-unify-browserwindow-isvisible-logic-cross-platform-1684230426706 branch from 940a233 to 02767e0 Compare May 16, 2023 20:58
@codebytere codebytere merged commit 912e211 into 23-x-y May 17, 2023
13 checks passed
@codebytere codebytere deleted the trop/23-x-y-bp-fix-unify-browserwindow-isvisible-logic-cross-platform-1684230426706 branch May 17, 2023 08:20
@release-clerk
Copy link

release-clerk bot commented May 17, 2023

Release Notes Persisted

Fixed an issue where BrowserWindow.isVisible() would incorrectly return true for minimized windows on Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
23-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant