-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add right click menu to song button label #83
Comments
Hi thanks for raising the issue. Can you mock up a screenshot or two showing what the issue is? I'm not really following what you are trying to explain. Many thanks in advance. |
ok thanks! much clearer. I'll have a think - I can't think of an immediate way to implement this. I'll have a closer look when I get some free time. |
this signal might do the trick - https://lazka.github.io/pgi-docs/#RB-3.0/classes/EntryView.html#RB.EntryView.signals.show_popup |
ok, I'm doubtful this will work in practise. Let me explain further. Say you are playing a song from the library view. Then you switch to the Play Queue. What is the right-click "browse by album" supposed to do? If you are playing and have filtered the list of songs by another browse by album or via a category - again it seems rather strange to have to reset the whole library view. Right-click stuff on labels is most unusual - people will not realise there is a right click available. Its inconsistent UI with other labels which don't have a right click. Probably #54 would cover this. Maybe have a couple of buttons on the popover which allows you to display just album tracks/artist tracks instead of the tracks before and after what is currently playing. |
Yeah, you are probably right. #54 looks very nice though. |
Would be be feasible? I'm talking about the right click menu that also appears when clicking on a song your list. Sometimes, I'm finding myself anywhere but near the current song that's playing. I'd be great to have the right click menu then to access options such as 'Browse this Album'.
The text was updated successfully, but these errors were encountered: