spot icon indicating copy to clipboard operation
spot copied to clipboard

Albums with multiple discs are unclear

Open lunaneff opened this issue 2 years ago • 12 comments

Describe the bug Some albums have multiple discs. For these albums, the track number jumps back to 1 in the middle of the album when a new disc starts.

To Reproduce Steps to reproduce the behavior:

  1. Go to any album with multiple discs (for example: Portal 2: Songs to Test By)
  2. Scroll down to the first track on the second disc

Expected behavior Spot should somehow show where a new disc starts. Maybe something like the way Spotify does it? (see screenshots)

Screenshots Current behavior:
image

How it's done on Spotify:
image

General information:

  • Distribution: Arch
  • Installation method: Flathub
  • Version: 0.3.1
  • Device used: Desktop

lunaneff avatar Jan 24 '22 13:01 lunaneff

Not very trivial technically I'm afraid - ListViews don't allow headers in between a list like that.

jannuary avatar Jan 25 '22 04:01 jannuary

Not very trivial technically I'm afraid - ListViews don't allow headers in between a list like that.

Maybe have two list views with a GtkLabel or something like that? I'm not too familiar with the codebase, but I feel like that could work.

Zaedus avatar Feb 02 '22 15:02 Zaedus

Maybe have two list views with a GtkLabel or something like that? I'm not too familiar with the codebase, but I feel like that could work.

No - listviews have to be the sole child of a scrollable and can't have stuff like headers 'in the way', so to speak, unfortunately

jannuary avatar Feb 02 '22 15:02 jannuary

We can try to do what Gnome Settings does with its separate sections in one page

Edit: grammar

Diegovsky avatar May 10 '22 13:05 Diegovsky

We can try to do what Gnome Settings does with its separate sections in one page

No we can't - completely different widgetry.

jannuary avatar May 10 '22 13:05 jannuary

So I did some search and it seems we can use AdwPreferencesGroup outside of a preferences page.

If it's not a good idea, maybe we could use AdwActionRow as a separator instead, or we could separate every disc in a AdwExpanderRow. What do you think?

Diegovsky avatar May 10 '22 15:05 Diegovsky

Again - completely different widgetry. Preferences* use ListBoxes - which are less scalable but are more flexible, which works great for a couple preferences. We use ListViews for playlists, which are more scalable (so that playlists and albums with >N tracks don't just break apart), but that adds restrictions: we have to have the playlist as the sole child of a scrolling view, so we can't have headers.

I've heard something about sections for ListViews being implemented, which might work in this case, but I do not know much more about that.

jannuary avatar May 10 '22 17:05 jannuary

Sorry, I totally mixed them up. Thank you for the explanation though.

Yeah, implementing sections for ListViews would be awesome for this, but we'll have to wait unfortunately

Diegovsky avatar May 10 '22 23:05 Diegovsky

@Zaedus are you still eager to contribute? I think I have an idea

Diegovsky avatar May 10 '22 23:05 Diegovsky

@Diegovsky sorry I don't have a lot of time anymore and I haven't worked with GTK in quite awhile. Wish I could've been of assistance.

Zaedus avatar May 11 '22 00:05 Zaedus

It's cool, I don't have the time to do it either unfortunately :)

Anyhow, if anyone wants to work on this I would be glad to mentor.

Diegovsky avatar May 11 '22 02:05 Diegovsky

It's won't be blocked by GTK whenever this lands: Draft: listview updates

jannuary avatar Aug 01 '22 04:08 jannuary