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: Fix play / pause icon to reflect current state #2713

Merged
merged 1 commit into from Oct 14, 2022

Conversation

cold-brewed
Copy link
Contributor

Summary

The main toolbar pause button always displays the pause icon, even when the system is paused. When paused, the button goes into the checked state (but remains a pause icon).

This PR changes it to:

  • Switch back and forth between play and pause icon depending on pause / run state
  • Update the tooltip based on the current state
  • Make the pause action button uncheckable so it will never appear checked or recessed (previous behavior, not necessary with the icon change)

Checklist

  • I have discussed this with core contributors already

References

N/A

@OBattler OBattler merged commit 932f2ce into 86Box:master Oct 14, 2022
@cold-brewed cold-brewed deleted the pause-icon-fix branch November 27, 2022 18:32
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.

None yet

2 participants