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

(SDL) Toggling off fullscreen should restore default window size #11991

Open
orbea opened this issue Apr 24, 2019 · 11 comments
Open

(SDL) Toggling off fullscreen should restore default window size #11991

orbea opened this issue Apr 24, 2019 · 11 comments
Labels
SDL2 Issue on SDL (or Qt in SDL code) but not all ports.
Milestone

Comments

@orbea
Copy link
Contributor

orbea commented Apr 24, 2019

What happens?

When the SDL fronted is started as fullscreen and the user attempts to toggle fullscreen from the menu ppsspp will change successfully become windowed, but the window size will still be fullscreen.

To reproduce:

  1. Start the SDL frontend with when fullscreen is already enabled.
  2. Toggle fullscreen in the menu.
  3. ppsspp will not be fullscreen, but the window will still cover the entire screen making it effectively fullscreen.

When started as a windowed program the original window size will (Probably?) not be the same size as when fullscreen and toggling fullscreen works out just fine.

This is followup to issue #11625 where fullscreen has been problematic with the SDL frontend for a long time.

What should happen?

When ppsspp is toggled from fullscreen it should make sure the window size and the fullscreen size are not the same. If they are the same it should automatically resize ppsspp to a smaller window. The default size for when starting ppsspp as a windowed program would suffice.

What hardware, operating system, and PPSSPP version? On desktop, GPU matters for graphical issues.

OS: Slackware64-current
ppsspp: d15a6cf
SDL: 2.0.9
GPU: AMD RX Vega 56 (amdgpu + radeonsi)

@unknownbrackets
Copy link
Collaborator

This issue is being closed as a duplicate of #11974.

This is an issue tracker - there's a forum if you want to discuss things. You wouldn't add another "bread" item to a grocery list every time you thought about it, right? We do the same here - we already have "bread" (this issue) on our list.

By keeping the list clean, we can also make sure all the information a developer needs to fix the issue is at their finger tips. When there are multiple issues, information gets lost and it becomes hard to fix things, so they are fixed more slowly. Duplicates won't make fixes appear faster.

And remember - PPSSPP is open source. Try to find new and useful information, and add it to the original issue. If you can't wait any longer, muster up your courage and try fixing it yourself. PPSSPP gets better the more people contribute. You could be that amazing person who solves this problem for everyone.

-[Unknown]

@orbea
Copy link
Contributor Author

orbea commented Apr 25, 2019

@unknownbrackets This is not a duplicate, please reopen. The issues are different.

@orbea
Copy link
Contributor Author

orbea commented Apr 25, 2019

@hrydgard Can you please reopen this issue? The linked issue is not remotely similar outside of both involving the fullscreen toggle. Its really discouraging when it seems that created issues are not even being read...

@hrydgard hrydgard reopened this Apr 25, 2019
@hrydgard
Copy link
Owner

Though, I have to say that they do sound very similar...

@orbea
Copy link
Contributor Author

orbea commented Apr 25, 2019

Thanks! Unlike that issue when the windows resize it seems to work as expected, I have a problem with how it will resize from fullscreen resolution to a windowed fullscreen resolution.

From my observations when toggling from fullscreen to windowed ppsspp will resize to the original window size it started with even if that is the same as fullscreen.

Additionally this is not specific to vulkan or any other video backend.

@orbea
Copy link
Contributor Author

orbea commented Apr 25, 2019

@hrydgard Do these images help you understand?

  • First ppsspp starts normally as a windowed program.
    windowed

  • I can toggle it fullscreen/windowed normally.
    fullscreen

  • If I close ppsspp while fullscreen and start it again it will still be fullscreen as expected. I can't resize the window since its fullscreen.

  • However if go to toggle fullscreen to windowed, its still fullscreen. Note the checkbox is not marked.
    windowed2

  • It really is windowed though, see how I can resize the window manually.
    windowed3

What should happen is that it resizes the window to be like the first picture instead of giving me a fullscreen window.

@vnctdj
Copy link
Contributor

vnctdj commented Apr 26, 2019

Ah, this last post definitely helps!
Then I feel like this issue should be renamed to something like "Toggling off fullscreen should restore default window size", no?

@orbea orbea changed the title (SDL) ppsspp does not always resize window when toggling fullscreen (SDL) Toggling off fullscreen should restore default window size Apr 26, 2019
@orbea
Copy link
Contributor Author

orbea commented Apr 26, 2019

Sure that works, done.

@unknownbrackets unknownbrackets added the SDL2 Issue on SDL (or Qt in SDL code) but not all ports. label Apr 27, 2019
@hrydgard hrydgard added this to the v1.10.0 milestone Aug 9, 2019
@hrydgard hrydgard modified the milestones: v1.10.0, Future Apr 28, 2020
@orbea
Copy link
Contributor Author

orbea commented Jun 25, 2020

I would like to also add that when the window is started with fullscreen disabled and the player toggles the fullscreen it mostly works at first, but after toggling it multiple times in a row the game resolution does not match the window size and you only see a portion of the content. This appears to be vulkan only.

Edit: Already fixed.

@hrydgard
Copy link
Owner

That latter part with the portion of the content should be fixed in the latest builds. It's not?

@orbea
Copy link
Contributor Author

orbea commented Jun 25, 2020

Yes you are right, I rebuilt the master (a0f8ad4) and that last part is now fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SDL2 Issue on SDL (or Qt in SDL code) but not all ports.
Projects
None yet
Development

No branches or pull requests

4 participants