musicat icon indicating copy to clipboard operation
musicat copied to clipboard

feat: icon view for smart playlists

Open daiyam opened this issue 10 months ago • 5 comments

This PR adds:

  • the album view for the smart playlists
  • new setting to select the preferred view (list by default)

daiyam avatar Jan 17 '25 15:01 daiyam

Oops, I forgot...

The view in the smart playlist use the new album_id attribute of a song, so a re-import is needed.

It's also only display the matching songs of an album.

daiyam avatar Jan 18 '25 14:01 daiyam

Interesting. Will this still set the queue to the full smart playlist, or just the selected album?

The view in the smart playlist use the new album_id attribute of a song, so a re-import is needed.

Need to think how to handle breaking changes like these going forward. Usually this would be via db migrations, but since the app is still in pre-release that's not strictly necessary. But I feel that a move from indexedDB to Sqlite will likely happen soon anyway.

basharovV avatar Jan 21 '25 08:01 basharovV

Also I think that the UI needs work to make sure that it all makes sense in various window sizes. Right now there is a lot of overlap when you resize the window down. So would need to answer questions like:

  • At very small breakpoints, do we still want to show the icon view, or revert back to the list view?
  • At which breakpoint do we hide the view options so it doesn't overlap with the rest of the UI.

basharovV avatar Jan 21 '25 09:01 basharovV

Interesting. Will this still set the queue to the full smart playlist, or just the selected album?

It will play the playlist on the context menu item Play the playlist It will play an album if you click on the play/pause button on the album cover

But I feel that a move from indexedDB to Sqlite will likely happen soon anyway.

#119

Tell me if I can make the changes

Right now there is a lot of overlap when you resize the window down.

Yep, I forgot about that. But even outside this PR, there are lot of bugs:

  • size up after a size up don't reposition all elements
  • in a small window, album view's controls should be moved to a single icon with a menu
  • maybe move lossy/lossless to a icon/menu
  • in a small window, when the sidebar is full, the queue is very small

I will try to fix those issues in another PR

daiyam avatar Jan 21 '25 18:01 daiyam

But even outside this PR, there are lot of bugs: in a small window, when the sidebar is full, the queue is very small

Yeah, there's a lot of elements so we have to be opinionated about which UI elements to show when. Maybe when resizing the window to a small width (~300px), hide all panels/popups except the queue. So even if the queue is hidden, force show it.

basharovV avatar Jan 22 '25 00:01 basharovV