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

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

Merged
ronjouch merged 2 commits into
nativefier:masterfrom
Adam777Z:master
Nov 7, 2022
Merged

Maximize window visual glitch on Windows fix (fix #1447)#1448
ronjouch merged 2 commits into
nativefier:masterfrom
Adam777Z:master

Conversation

@Adam777Z

@Adam777Z Adam777Z commented Aug 8, 2022

Copy link
Copy Markdown
Contributor

@ronjouch

ronjouch commented Aug 29, 2022

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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
Comment thread app/src/components/mainWindow.ts
Comment thread app/src/components/mainWindow.ts Outdated
@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