[GH-ISSUE #528] Queue key opens multiple popups #303

New issue

Closed

opened 2026-03-02 23:46:21 +03:00 by kerem · 2 comments

kerem commented

2026-03-02 23:46:21 +03:00

Owner

Originally created by @m-torhan on GitHub (Jul 31, 2024).
Original GitHub issue: https://github.com/aome510/spotify-player/issues/528

Describe the bug
After pressing "z" multiple times to open queue, each key down opens new popup and it is required to press "backspace" equal number of times to close all of them.

To Reproduce

Press "z" multiple times.
Press "back" - the queue popup is still visible.

Expected behaviour
Only one popup is opened / second press closes it.

Environment

OS: Linux
Application version: 0.19.1

Originally created by @m-torhan on GitHub (Jul 31, 2024). Original GitHub issue: https://github.com/aome510/spotify-player/issues/528 **Describe the bug** After pressing "z" multiple times to open queue, each key down opens new popup and it is required to press "backspace" equal number of times to close all of them. **To Reproduce** 1. Press "z" multiple times. 2. Press "back" - the queue popup is still visible. **Expected behaviour** Only one popup is opened / second press closes it. **Environment** - OS: Linux - Application version: 0.19.1

kerem closed this issue

2026-03-02 23:46:22 +03:00

kerem commented

2026-03-02 23:46:23 +03:00

Author

Owner

@aome510 commented on GitHub (Jul 31, 2024):

queue is updated to be a page instead of a popup a while ago. We might want to implement a support for overriding page history if the current page is the same as the last one, but I'm not sure how useful this feature will be

@aome510 commented on GitHub (Jul 31, 2024): `queue` is updated to be a page instead of a popup a while ago. We might want to implement a support for overriding page history if the current page is the same as the last one, but I'm not sure how useful this feature will be

kerem commented

2026-03-02 23:46:23 +03:00

Author

Owner

@m-torhan commented on GitHub (Aug 1, 2024):

@aome510 okay, I see. Now it makes more sense as the queue is a page. Maybe instead of overriding history, we could ignore goto if the target page is the same as current?

For me, it is okay like it is now. Feel free to close the issue.

@m-torhan commented on GitHub (Aug 1, 2024): @aome510 okay, I see. Now it makes more sense as the `queue` is a page. Maybe instead of overriding history, we could ignore goto if the target page is the same as current? For me, it is okay like it is now. Feel free to close the issue.

kerem referenced this issue

2026-03-02 23:49:28 +03:00

[PR #303] [MERGED] support disabling notification from config #707

kerem referenced this issue

2026-03-14 15:25:10 +03:00

[PR #303] [MERGED] support disabling notification from config #1643