bmm-web icon indicating copy to clipboard operation
bmm-web copied to clipboard

Visible playlist name

Open leandergangso opened this issue 1 year ago • 10 comments

It would be nice to make the name for each playlist visible.

It might be fine for some of the playlists like here

image

but it's more confusing here

image

leandergangso avatar Jun 16 '24 21:06 leandergangso

Yeah, that seems to be something to consider.

We could maybe try adding the title just below the covers? Or maybe it could show up as an overlay when hovering the covers.

sifferhans avatar Jun 24 '24 13:06 sifferhans

Here are 2 suggestions I was thinking about (if we don't want to just put the title bellow the img?):

image

leandergangso avatar Jun 24 '24 13:06 leandergangso

@sclausendk pinging you in case you're interested

andreasgangso avatar Jun 24 '24 14:06 andreasgangso

@leandergangso The third option is basically how it works on the old website, could work

sifferhans avatar Jun 24 '24 19:06 sifferhans

I'll design something. For all recent playlists and albums, it will be duplicate though. So I would rather update the few covers that does not include a title in the image. But I'll explore it! Thanks for the suggestions!

sclausendk avatar Jun 24 '24 20:06 sclausendk

@sclausendk One disadvantage with putting the text in the cover is that it can't be translated. Having the title separate, as before, allows to easily provide different translations (Live sanger, Live songs, Live-Aufnahmen, Chansons en direct, ...).

Not sure if that outweighs the positives, but wanted to point that out.

kkuepper avatar Jul 04 '24 16:07 kkuepper

Good point.

Idea: If you find a nice clear font, drop text on covers, then this text can be placed on top of the cover, making it similar to how it is currently. Can then be translated no problem.

leandergangso avatar Jul 04 '24 16:07 leandergangso

@leandergangso @kkuepper I see your point. But this results in all covers being simple images, which doesn't work well in the player and OS players outside of the app. It also makes it difficult to show logos from conferences, podcasts, bible studies etc.

Putting text on top of images almost always results in A LOT of issues with legibility, contrast, scaling, duplicate text in image and UI. Considering that most playlists and albums has text in them today, I lean towards fixing the covers that do not have text.

I'll explore how we can handle localisation better. There might be some creative solutions here 🤔

sclausendk avatar Jul 05 '24 08:07 sclausendk

I'd argue that some of the examples pointed out here are language agnostic, and those playlists can have text in the cover image, since all the content is aimed at a group of speakers of a certain language.

E.g. I argue, that the playlist "Sanger" aims for a norwegian audience, even though there are english songs in there. Having the same cover for "Live sanger" seems unlucky, but having a different cover for the albums "Kommentarutgave Romerbrevet kap. 6-8" and "Commentary to Paul´s Letter to the Romans" seems doable to me. Those two, in my opinion, should also remain separate and not as translated like e.g. meetings, because they have different timestamp for references like bible-verses - just wanted to mention this argument if someone asks why they are uploaded as separate albums.

One example where the text has been embedded for language-agnostic playlists is "Smile" and "Smil", whereby I in this case am often confused which one is which language, because the difference is just about one letter 😆 So a hint to the language (as we had on the old website) might be worth an idea:

Screenshot 2024-07-05 at 11 42 45

SimonSimCity avatar Jul 05 '24 09:07 SimonSimCity

@SimonSimCity I think the screenshot showcases what I want to avoid. Duplicate titles for all covers. Overlays on image covers up important details. Overlays are generally distracting and messy. Flags everywhere :D

sclausendk avatar Jul 05 '24 11:07 sclausendk

@sclausendk What's the solution to this issue? We're getting complaints about this.

kkuepper avatar Dec 11 '24 16:12 kkuepper

@kkuepper I suggest we implement a temporary fix until Søren comes up with a proper solution.

We can eg. do this, just a tad prettier:

CleanShot 2025-01-09 at 13 48 52

Or we can go the same route as in the mobile app, and just put the title below the cover

sifferhans avatar Jan 09 '25 12:01 sifferhans

@sclausendk Could you take a quick look at this and shout if it's too ugly / noisy? If it looks ok, I think it could be a quick solution until you design something specific

https://github.com/bcc-code/bmm-web/pull/575

Image

sifferhans avatar Mar 20 '25 09:03 sifferhans

@sifferhans With center alignment, I'm happy with it :+1: Potentially with truncation to one line?

sclausendk avatar Mar 20 '25 10:03 sclausendk

@sclausendk I've truncated to two lines now, can do one line 👍 A single line is more predictable for sure

Image Image

sifferhans avatar Mar 20 '25 10:03 sifferhans

One line makes it more consistent 👍

sclausendk avatar Mar 20 '25 10:03 sclausendk