Skip to content

Conversation

@rockhero1234
Copy link
Contributor

No description provided.

@rockhero1234 rockhero1234 changed the title separate episode label from title & setting for it feat:separate episode label from title & setting for it Oct 29, 2025
if (card.type?.isMovieType() == false && showEpisodeLabel) {
val context = cardText?.context
val text = when {
showTitle -> context?.getNameFull(card.name, card.episode, card.season)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not understand this line? If you disable title but enable episode label it looks very weird with no labels on movies but labels on tv-series. Futhermore this breaks the UI as it will push the UI up and down moving everything, if you have mixed set of both movies and tv-series.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested working fine for me

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In your continue watching have a row like this where you only have 3~4 items visible at the same time.

"MMMMMEEEEEEMEMEMEMEE" where M = Movie and E = Episode. If you only have movies then it is hidden, but as soon as you scroll to episodes it pops in and moves the entire layout down due to wrap_content. Moving layouts after inflating it is a no-go for UX, and is only done in Cloudstream when it is really needed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants