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) The fullscreen keybind toggle does not work in the menu #11992

Closed
orbea opened this issue Apr 24, 2019 · 10 comments
Closed

(SDL) The fullscreen keybind toggle does not work in the menu #11992

orbea opened this issue Apr 24, 2019 · 10 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 using the SDL frontend the fullscreen keybind toggle does not work.

In Settings -> Controls -> Control mapping -> Toggle fullscreen I set it to F11 where it shows kbd.F11 and found it does nothing. I next set it to the f key (kbd.F) and it still does nothing.

The menu toggle however does work while the Qt frontend which has the opposite issue where the menu toggle does not work and the keybind toggle does work with F11. (Issue #11626).

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

What should happen?

The fullscreen keybind toggle should work in the SDL frontend.

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

Is this different from #11974? It sounds like you get entirely different behavior which is possibly distro or window manager specific?

-[Unknown]

@unknownbrackets unknownbrackets added the SDL2 Issue on SDL (or Qt in SDL code) but not all ports. label Apr 25, 2019
@orbea
Copy link
Contributor Author

orbea commented Apr 25, 2019

Yes, both of my two recent issues are different than that issue. I can't reproduce that issue with the master since it was fixed recently for me.

@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
@vnctdj
Copy link
Contributor

vnctdj commented Jun 24, 2020

I can't reproduce this (not sure if I have ever reproduced it though).
What about you @orbea?

@orbea
Copy link
Contributor Author

orbea commented Jun 25, 2020

Yes, its broken here still...

@hrydgard
Copy link
Owner

In game or in the menu? Works for me in-game but not in the PPSSPP menu screens.

Though, that probably should be fixed.

@orbea
Copy link
Contributor Author

orbea commented Jun 25, 2020

I see what you mean, but its kind of both.

  • Like you say, in the menu its entirely ignored.
  • If I start ppsspp with fullscreen disabled and then start a game, it works fine.
  • If I start ppsspp with fullscreen enabled and then start a game, I can see that it leaves fullscreen mode, but the it never resizes the window accordingly like when its started without fullscreen.

I think this needs two fixes.

  1. It should work in the menu as well as in game.
  2. When started fullscreen it should still know the size of the window when not fullscreen in case the player toggles it.

Edit: I tested other SDL2 programs and they do not have the second problem, so it should be fixable.

@hrydgard
Copy link
Owner

Aha! That's a much better report, thank you, now it makes sense. Yeah in this case it should avoid remembering the fullscreen window size indeed.

@hrydgard hrydgard modified the milestones: Future, v1.11.0 Jun 25, 2020
@orbea
Copy link
Contributor Author

orbea commented Jun 25, 2020

@hrydgard I now remember I made a second issue for the fullscreen size here. #11991

@orbea orbea changed the title (SDL) The fullscreen keybind toggle does not work (SDL) The fullscreen keybind toggle does not work in the menu Jun 25, 2020
@vnctdj
Copy link
Contributor

vnctdj commented Jun 25, 2020

So... This issue can now be closed as a duplicate of #7542 :)

@hrydgard
Copy link
Owner

Duplicate of #7542.

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