search

Taiko2k / Tauon

The music player of today! :city_sunrise:
https://tauonmusicbox.rocks
GNU General Public License v3.0
1.77k stars 67 forks source link

[Feature request] auto name playlists #41

Closed Solarunit closed 5 years ago

Solarunit commented 5 years ago

It would be nice to auto-generate names for the playlists when adding folders/files. E.g. if there is only one artist in the playlist, playlist should be named by artist's name. If we add folder containing various artists, playlist should be named by the folder's name.

Taiko2k commented 5 years ago

Sounds interesting, though, I wonder what the condition for this should be?

Taiko2k commented 5 years ago

Also, what should happen if the user later adds albums that are not related to existing tracks?

Solarunit commented 5 years ago 
 If a playlist was never named by the user (seem like maybe a good choice to me)

Agree.

Also, what should happen if the user later adds albums that are not related to existing tracks?

imho nothing, auto name should only work once. Or it can be optional.
(AIMP player I think changes auto-name if the user adds albums later)

Taiko2k commented 5 years ago

Yes, just applying once seems like the sane method.

I have an implementation running which always tries to name a new playlist with the following priority:

  1. albumartist - album
  2. artist - album
  3. Artist: artist
  4. The name of the highest common folder

I think this works well. This should solve the issue where my list is often filled with all "New Playlist (5)" etc since naming manually is inconvenient.

Solarunit commented 5 years ago

Nice!

Solarunit commented 5 years ago

tauon3.6.X1.flatpak quick test - works very well! But when we have the 3 case (Artist: artist) - the "Artist" is also part of the name.

Taiko2k commented 5 years ago

That was intentional.

My reasoning was that having "Artist:" makes it clear what it is, where the format " xxx - xxx " already implies artist - album. Maybe its unnecessary to do this?

Solarunit commented 5 years ago

I think it should be just an artist . Because you would never name playlist manually as "Artist: Coldplay"

Taiko2k commented 5 years ago

Actually another reason I chose that was because its also the format the search results have, but I think you're right, that doesn't perhaps make as much sense here.