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

[VDG] Do not call SetWindowSize when windows state is Maximized #9185

Merged

Conversation

wieslawsoltes
Copy link
Collaborator

@wieslawsoltes wieslawsoltes commented Sep 26, 2022

Fixes: #8985
Fixes: #8986

soosr
soosr previously approved these changes Sep 26, 2022
Copy link
Collaborator

@soosr soosr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer this solution, just because it is simpler.

utACK.

Copy link
Collaborator

@yahiheb yahiheb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes #8986 (I am sure about login button is not fully visible)
But it does not fix #8985

@soosr
Copy link
Collaborator

soosr commented Sep 27, 2022

But it does not fix #9185

You mean #8985?

Copy link
Collaborator

@soosr soosr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK, it fixes both issues on windows.

One thing to mention... when my window opens at maximized. The top of the window is cut down a tiny bit. But this is not related to this PR I guess.

image

@soosr
Copy link
Collaborator

soosr commented Sep 27, 2022

I have to test this on macOS as well because AFAIK the window doesn't know the screen until it is shown there. So the calculations in SetWindowSize would never happen.

@soosr
Copy link
Collaborator

soosr commented Sep 27, 2022

I have to test this on macOS as well because AFAIK the window doesn't know the screen until it is shown there. So the calculations in SetWindowSize would never happen.

Tested, it works as expected.

@yahiheb
Copy link
Collaborator

yahiheb commented Sep 27, 2022

But it does not fix #9185

You mean #8985?

Yes it does not fix that issue for me. WindowWidth & WindowHeight do not change when you maximize Wasabi window.

One thing to mention... when my window opens at maximized. The top of the window is cut down a tiny bit. But this is not related to this PR I guess.

Same here.

@soosr
Copy link
Collaborator

soosr commented Sep 27, 2022

Yes it does not fix that issue for me. WindowWidth & WindowHeight do not change when you maximize Wasabi window.

It does not even have to change it in maximized state. Already discussed here #8985

@pull-request-size pull-request-size bot added size/L and removed size/M labels Sep 27, 2022
yahiheb
yahiheb previously approved these changes Sep 27, 2022
Copy link
Collaborator

@yahiheb yahiheb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened a new issue #9193 for #9185 (review)

tACK

@wieslawsoltes
Copy link
Collaborator Author

tACK, it fixes both issues on windows.

One thing to mention... when my window opens at maximized. The top of the window is cut down a tiny bit. But this is not related to this PR I guess.

image

This is different issue on Windows

Copy link
Collaborator

@yahiheb yahiheb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

@soosr soosr merged commit 90a7949 into zkSNACKs:master Sep 27, 2022
@wieslawsoltes wieslawsoltes deleted the vdg/DoNotCallWindowSizeWhenMaximized branch October 3, 2022 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Window's controls didn't get update WindowWidth & WindowHeight do not change when you maximize Wasabi window
3 participants