-
-
Notifications
You must be signed in to change notification settings - Fork 466
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Display length of show/movie in hour minute format (like web client) #3878
base: master
Are you sure you want to change the base?
Conversation
32c36d3
to
768cd0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't modify translation files (values-xx folders) directly. Those are automatically managed by our translation tool.
app/src/main/java/org/jellyfin/androidtv/ui/itemdetail/FullDetailsFragment.java
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/androidtv/ui/itemdetail/FullDetailsFragment.java
Outdated
Show resolved
Hide resolved
As requested, in 88ad0e0 I reverted the translation files, so this PR won't change them directly. |
Co-authored-by: Niels van Velzen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code doesn't compile
Hey @noahvogt, are you still interested in finishing this pull request? it's currently waiting for you to address my comment above. |
Display length of show/movie in hour minute format
Changes
Currently the length of shows/movies shown in FullDetailsFragment.java is in minutes only, or to say as a regex
[0-9]+ min
. With my changes, it now shows in the format[0-9]+h [0-9][0-9]?m
. Note that the minutes are not zeropadded (displaying eg2h 02m
instead of2h 2m
) as this is the way the web client does it as well.For more consistency with the main web client, I decided to just change it instead of adding an option so that the user can switch between both display methods. But if that's prefered, I can change it before resubmitting.
Issues
Fixes #2806