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

Remove "Remember popup properties" toggle and make it the default behaviour #4329

Closed
opusforlife2 opened this issue Sep 27, 2020 · 3 comments · Fixed by #4367
Closed

Remove "Remember popup properties" toggle and make it the default behaviour #4329

opusforlife2 opened this issue Sep 27, 2020 · 3 comments · Fixed by #4367
Assignees
Labels
good first issue Easy/simple issues perfect for newcomers to get involved in the project GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background)

Comments

@opusforlife2
Copy link
Collaborator

Opened after discussion on IRC.

There doesn't seem to be much use for a case where the user would prefer that the app doesn't remember popup size and position. So the default behaviour should be for the app to remember, and the setting should be removed entirely.

@opusforlife2 opusforlife2 added GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background) labels Sep 27, 2020
@TobiGr TobiGr added the good first issue Easy/simple issues perfect for newcomers to get involved in the project label Sep 27, 2020
@panoreak
Copy link
Contributor

panoreak commented Oct 1, 2020

Hello! Can I work on this?

@opusforlife2
Copy link
Collaborator Author

Absolutely! Go ahead.

@DI555

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Easy/simple issues perfect for newcomers to get involved in the project GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants