-
Notifications
You must be signed in to change notification settings - Fork 3k
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
win32: remove SC_RESTORE handling #16127
base: master
Are you sure you want to change the base?
Conversation
We don't support other ways of fullscreening. Fixes: mpv-player#16119
It doesn't seem to be neeeded anymore.
@na-na-hi do you know where this would be needed? I couldn't reproduce on my end. Maybe it was FSE mode that is no longer with us. |
Download the artifacts for this pull request: |
After a maximized window is put in fullscreen, when you press win+down, it exits fullscreen in a maximized state. |
I see, but what is expected behavior in this case? Shouldn't it exit fullscreen? Same as win+down goes from maximized to windowed first. |
It does. As I already said, currently win+down in fullscreen exits fullscreen if |
But if it is not maximized it minimize instead. Is this expected to you? I found dozen other bugs when testing this again, so we can postpone this discussion. I only wanted to know if this is expected that |
win+down on a non-maximized window normally minimizes it. Technically I don't think there is any expected behavior in this case. As long as it does not leave the window in a buggy state it should be fine. |
No description provided.