Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Maximize window visual glitch on Windows fix (fix #1447) #1448

Merged
merged 2 commits into from
Nov 7, 2022
Merged

Maximize window visual glitch on Windows fix (fix #1447) #1448

merged 2 commits into from
Nov 7, 2022

Conversation

Adam777Z
Copy link
Contributor

@Adam777Z Adam777Z commented Aug 8, 2022

@ronjouch
Copy link
Contributor

ronjouch commented Aug 29, 2022

@Adam777Z hi and thanks for the PR. Can you precise what you mean by "glitch"? A precise description or screenshot/screencast would be welcome.

Also, glitch or not, your PR is a functional regression, and will I think annoy users used to the current behavior. I think I'd rather keep the glitch but honor the desired behavior. Waddayathink?

@Adam777Z
Copy link
Contributor Author

This is only a temporary fix until it's fixed in Electron.

There's no change in functionality, it works as before, just without the visual glitch on Windows. This change is for Windows only.

You can test this easily on Windows:

  1. Open an app made with the latest version of Nativefier using the default options
  2. Maximize the window
  3. Close the app
  4. Open the app again. Because the window size was saved, it should open maximized, and the visual glitch can be seen without this fix applied.

@ronjouch ronjouch changed the title Maximize window visual glitch on Windows fix Maximize window visual glitch on Windows fix (fix #1447) Nov 7, 2022
@ronjouch ronjouch linked an issue Nov 7, 2022 that may be closed by this pull request
5 tasks
app/src/components/mainWindow.ts Show resolved Hide resolved
app/src/components/mainWindow.ts Outdated Show resolved Hide resolved
@ronjouch ronjouch merged commit f046b61 into nativefier:master Nov 7, 2022
Adam777Z added a commit to Adam777Z/nativefier that referenced this pull request Nov 9, 2022
…nativefier#1448)

Fix for [Maximize window visual glitch on Windows · Issue nativefier#1447 · nativefier/nativefier](nativefier#1447)

Co-authored-by: Ronan Jouchet <ronan@jouchet.fr>
Adam777Z added a commit to Adam777Z/nativefier that referenced this pull request Nov 9, 2022
…nativefier#1448)

Fix for [Maximize window visual glitch on Windows · Issue nativefier#1447 · nativefier/nativefier](nativefier#1447)

Co-authored-by: Ronan Jouchet <ronan@jouchet.fr>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maximize window visual glitch on Windows
2 participants