Enabled all titles to show under poster images

[candry7731]

#365 - Enable Titles to show when grid is loaded instead of on focus...

Changes
Set scrolling label to visible

[neilsb]

This looks good, but I'd like to hold off this until we get the client side setting place, so a user can choose to switch whether to have them on or off in the preferences.

I'll make a concerted effort to get that in place this week (sorry for the delay), and it'll be a trivial change to check that in this PR.

[neilsb]

I've hooked up a user setting for this, now the settings screen is in place.

I've had a play with your changes, and have a couple of things for discussion regarding long titles

• Item titles are currently Scrolling Labels. When they're are always displayed, do we want them always scrolling, or should they only scroll when the item is highlighted?
• When you've got a couple of items with long titles together, the titles now run into each other (especially if scrolling). I think it may be good to do something to address this.

[candry7731]

I've hooked up a user setting for this, now the settings screen is in place.

I've had a play with your changes, and have a couple of things for discussion regarding long titles

• Item titles are currently Scrolling Labels. When they're are always displayed, do we want them always scrolling, or should they only scroll when the item is highlighted? I set this to only scroll when item has focus. I think this works better
• When you've got a couple of items with long titles together, the titles now run into each other (especially if scrolling). I think it may be good to do something to address this. I updated the text width to be smaller and this this gives more padding left and right

[candry7731]

Sounds good! Just trying to implement some of the feature requests.

…

On Sun, Apr 24, 2022, 2:38 PM Neil Burrows ***@***.***> wrote: This looks good, but I'd like to hold off this until we get the client side setting place, so a user can choose to switch whether to have them on or off in the preferences. I'll make a concerted effort to get that in place this week (sorry for the delay), and it'll be a trivial change to check that in this PR. — Reply to this email directly, view it on GitHub <#570 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AT6PDRCI2XO352HQ2SJ3OGDVGWPKFANCNFSM5T2TVXRQ> . You are receiving this because you authored the thread.Message ID: ***@***.***>