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

Qt: Disables the Pause and Stop button when no game is launched #2431

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Xphalnos
Copy link
Collaborator

When no game is running, the Pause and Stop buttons are unclickable.

2025-02-14.15-17-23.mp4

@georgemoralis
Copy link
Collaborator

i think it is better to be disabled not unchickable

@Xphalnos
Copy link
Collaborator Author

i think it is better to be disabled not unchickable

What do you mean? That the buttons are always disabled?

@georgemoralis
Copy link
Collaborator

pause and reset should be disable if a game isn't running

@Xphalnos
Copy link
Collaborator Author

pause and reset should be disable if a game isn't running

You mean they only appear when a game is launched?
If so, I'm afraid they'll break the interface when they appear.

@GHU7924
Copy link

GHU7924 commented Feb 17, 2025

I suggest switching to Pause for the Play button.
That is, you press Play, the game starts, and the Play button changes its appearance and functionality to Pause. (Play -> Pause)
If you press Pause, the game pauses, and the Pause button changes its appearance and functionality to Play. (Pause -> Play)
One button switches its functionality between Play and Pause. (Play -> Pause -> Play)

Reset (Restart), if possible, so that it works in two modes:

  1. If you are in the game, it restarts the game.
  2. If you are not in the game, then it runs the last game you played.

You can read some of my thoughts about these buttons in this #2441 (comment) .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants