Skip to content

Localized formatting for season and episode display - #1884

Open
tal-sarid wants to merge 5 commits into
damontecres:mainfrom
tal-sarid:fea/localized-season-and-episode
Open

Localized formatting for season and episode display#1884
tal-sarid wants to merge 5 commits into
damontecres:mainfrom
tal-sarid:fea/localized-season-and-episode

Conversation

@tal-sarid

Copy link
Copy Markdown
Contributor

Description

Added localized formatting for season/episode numbers across the UI because it was in English-only in every UI language.

Related issues

None

Testing

Tested on a physical Google TV Streamer 4K device. Checked manually relevant screens while browsing and during playback.

Screenshots

No frontend changes are visible in English and new translations will be contributed through Weblate for other languages later, as usual.

AI or LLM usage

Generated initial code with AI, then manually passed over it while cleaning and refactoring where needed.

@damontecres damontecres left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This looks pretty good!

Just need to switch from LocalContext to LocalResources in a few places.

Comment thread app/src/main/java/com/github/damontecres/wholphin/ui/detail/livetv/Components.kt Outdated
Comment thread app/src/main/java/com/github/damontecres/wholphin/ui/detail/livetv/Components.kt Outdated
Co-authored-by: Damontecres <damontecres@gmail.com>
@tal-sarid
tal-sarid requested a review from damontecres August 30, 2026 17:50
@tal-sarid

Copy link
Copy Markdown
Contributor Author

I fixed the unit tests by falling back to default format strings when WholphinApplication.instance in uninitialized.

Also, I noticed that I had a missing case for hours+seconds localized duration format and fixed it but if you think it is totally unrelated here I can revert it and open a different PR for that.

@damontecres damontecres left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good to me!

Also, I noticed that I had a missing case for hours+seconds localized duration format and fixed it but if you think it is totally unrelated here I can revert it and open a different PR for that.

It's small and related-ish, so I think it's fine!

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