Skip to content
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

Embedded cover art for tracks with album "Soundcloud" is ignored in some players #184

Open
justsh opened this issue Apr 10, 2017 · 4 comments

Comments

@justsh
Copy link

justsh commented Apr 10, 2017

Possibly similar to #183

In several players that I use, the image chosen as the Album Art is not automatically the embedded Cover image, but instead is chosen based on the name of the Album. If no image can be found for the album, the Album Art is left as a placeholder, otherwise, the first matching Cover will be applied to all images with that Album name (regardless of Artist).
Why this is the case is beyond me.

Anyway, I so far have seen this problem on tracks downloaded that did not have Album tags, and therefore ended up with "Soundcloud" as the album name.
The embedded image for the first track with the Album "Soundcloud" ends up showing as the Album Art for all tracks with the Album name "Soundcloud".
The only real workaround for this I've found is to set the Album field to "%Title [Soundcloud]" (or something else unique), since most of the tracks I'm grabbing are free singles with no real album name anyway.

@flyingrub
Copy link
Collaborator

I had the album art issue with a lot of player on android but liked to have all my soundcloud track in a soundcloud album. Now I use poweramp and I don't have this problem anymore.

I don't mind removing the album field for the track that are not downloaded from a playlist. But I would like to get some other people opinion on this. Do they prefer to have all their track in one album ? or have all the correct album art ?

@justsh
Copy link
Author

justsh commented Apr 19, 2017

Well, removing the album field would have the same result in that all all songs would fall under the "Unknown" album, or whatever the player's default is. It might be nice to have the option to do either for songs not in a playlist that also have empty album fields: either use just "Soundcloud" by default, or use "{title} [Soundcloud]". That way the current behaviour is also preserved.

@hXtreme
Copy link

hXtreme commented Apr 19, 2017

I recommend that if we don't get Album information then we set the album field as "{title}" and then save the image as "{title}.jpg". The reason for the recommendation is that when the files are in sorted order the cover and track will be close together.

This may be a separate issue but we can also add a default image for tracks that don't have a cover, probably reading "SCDL" in some neat font.

@flyingrub
Copy link
Collaborator

We could effectively keep the current behavior and add an option to use "{title} [Soundcloud]" as album.

@hXtreme we currently do not keep the image as a .jpeg file. This could be another issue, feel free to open it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants