spot icon indicating copy to clipboard operation
spot copied to clipboard

[Feature] Add context menus

Open Douile opened this issue 4 years ago • 5 comments

Add context menus to items. e.g. playlist copy URL or spotify URI. This is aimed to make navigation easier. I have included lists of things I think each context menu should contain. This is definitely not essential so should be low priority imo.

Songs:

  • View artist(s) (note. songs and albums can have multiple artists)
  • View album
  • Copy link
  • Copy URI

Albums:

  • View artist(s)
  • Copy link
  • Copy URI

Artists:

  • Copy link
  • Copy URI

Playlists:

  • View owner
  • Copy link
  • Copy URI

Douile avatar Feb 23 '21 19:02 Douile

There are some context menus already actually, that can be added by any view that uses Playlist, for instance the Now playing view :) I intend to add more contextual actions in the future but feel free to contribute some in the meantime!

xou816 avatar Feb 24 '21 08:02 xou816

I have started working on this in https://github.com/Douile/spot/tree/patch/menus

Douile avatar Feb 28 '21 14:02 Douile

Would like it if these menus could be accessed by right clicking the row, I am looking into the best way to do this (for accessibility reasons should also support context menu key).

Would also like to be able to right click playlist and album art (and maybe artist names) to be able to access similar context menus but this seems more complicated and I feel should be a separate PR to #111.

Douile avatar Feb 28 '21 21:02 Douile

Putting this here for my own reference: https://developer.gnome.org/gtkmm-tutorial/stable/sec-treeview-examples.html.en#treeview-popup-menu-example

@xou816 how do you feel about right click popup context menus?

Douile avatar Mar 04 '21 13:03 Douile

No objection, but probably to use actual old style menus but rather popovers :)

xou816 avatar Mar 04 '21 16:03 xou816