sct / overseerr

Request management and media discovery tool for the Plex ecosystem
https://overseerr.dev
MIT License
3.78k stars 440 forks source link

feat(usersettings): add ability to hide tags on details page #3810

Open jordanjones98 opened 6 months ago

jordanjones98 commented 6 months ago

fix #3350

Description

Add ability for a user to collapse tags on the movie/tv detail page from a checkbox. I made the text the same size as the season text, maybe it should be smaller? Up to do whatever there.

[question] This is at the user level right now, because that is where the other PR had it, should it be a server level setting?

Screenshot (if UI-related)

Collapse Tags checkbox in user settings page image

Collapse Tags checkbox unchecked (default) image

Collapse Tags checkbox checked image

To-Dos

Issues Fixed or Closed

sct commented 6 months ago

What about instead of just hiding it entirely, we add an option to auto collapse the tags? That way users could still see them if they want to?

jordanjones98 commented 6 months ago

What about instead of just hiding it entirely, we add an option to auto collapse the tags? That way users could still see them if they want to?

Yeah I like that idea, I can implement that.

jordanjones98 commented 6 months ago

What about instead of just hiding it entirely, we add an option to auto collapse the tags? That way users could still see them if they want to?

I updated it, I am probably going to pull out UI into it's own component so it is in one spot, but that is a tomorrow problem. Functionality and UI is there right now.

jordanjones98 commented 6 months ago

@sct Updated to use a shared component. The only configurable thing besides the tags is the type of movie or tv in the URL. I think just a prop with those two options is safe, but if you would rather do it a different way let me know. That migration should be fixed now too.