R-a-dio / valkyrie

R/a/dio software stack
https://r-a-d.io
MIT License
5 stars 3 forks source link

themes: Hover to reveal isn't very good #134

Open Bevinsky opened 3 months ago

Bevinsky commented 3 months ago

A lot of elements have overflow:hidden and text-overflow:ellipsis. I'm guessing this is what makes text hidden when it's too long and then appear when it's hovered over.

I don't think this is good design; it makes the UI jump around a lot when hovering over elements like LP and queue on the main page. It's better to just fill the box and wrap, making the box vertically larger. I guess it solves the issue with song titles that don't break properly and then escape the box boundary, but there must be better solutions to that than hiding most of the content at all times.

One place where it's also causing render issues seems to be the DJ name. Part of the text is cut off at the bottom when not hovering: This is obviously minor, but it bothered me.