App strings reworked 4 #950
Conversation
|
Piped summary etc is shown by long pressing the Piped card in the about section |
app/src/main/res/values/strings.xml
Outdated
| <string name="auth_instance">Authentication instance</string> | ||
| <string name="auth_instance_summary">Use a different instance for authenticated calls.</string> | ||
| <string name="watch_positions_summary">Continue from last playback position</string> | ||
| <string name="auth_instance">Log in elsewhere</string> |
There was a problem hiding this comment.
We should better leave that Authentication instance
There was a problem hiding this comment.
The benefit escapes me.
For reasons why this may be the official term, it is also the least friendly string.
It seems the actual meaning alludes people not already technically adept.
Albeit a setting for technical users, it is also important that regular ones understand not to use it, and more importantly aren't alarmed by its presence.
The question that arises is whether an authentication instance is needed in its own right.
This might lead people to try to input something, not knowing that there is already a sane default.
If the user doesn't know what "authenticated calls" are, in all likelihood it could be that the user thinks
that is an extra benefit available when connecting to a "different instance" to do so.
The direct translations aren't any better, and the attempted ones are more misleading.
They are also longer.
app/src/main/res/values/strings.xml
Outdated
| <string name="auth_instance_summary">Use a different instance for authenticated calls.</string> | ||
| <string name="watch_positions_summary">Continue from last playback position</string> | ||
| <string name="auth_instance">Log in elsewhere</string> | ||
| <string name="auth_instance_summary">Uses a different instance.</string> |
app/src/main/res/values/strings.xml
Outdated
| <string name="pure_theme">Pure theme</string> | ||
| <string name="pure_theme_summary">Pure white/black theme</string> | ||
| <string name="pure_theme_summary">Only black and white</string> |
There was a problem hiding this comment.
I suggest keeping the old term here too (Pure white/black theme)
There was a problem hiding this comment.
My thinking is that "Black and white" is a natural linguistic concept.
Moreover, it could be just "Black and white" in the title, without any summary.
This again begs the question of why it isn't a selection in the theme alternatives to begin with.
Because it isn't a black and white version of the selected theme(?)
Also both require an app restart, so no loss(?) making it one category.
On my device, the Appearance section scrolls by one entry.
Doing away with the toggle would help if not fix that completely.
app/src/main/res/values/strings.xml
Outdated
| <string name="update_now">Do you want to update the app now?</string> | ||
| <string name="seekbar_preview">Seekbar preview</string> | ||
| <string name="seekbar_preview_summary">Preview the video by seeking to the position when scrubbing the seekbar.</string> | ||
| <string name="update_now">Install the new LibreTube now?</string> |
There was a problem hiding this comment.
"Install the new LibreTube version now?" might be better
app/src/main/res/values/strings.xml
Outdated
| <string name="checking_frequency">Checking frequency</string> | ||
| <string name="new_streams_count">%1$s new streams are available</string> | ||
| <string name="new_streams_by">New streams by %1$s …</string> | ||
| <string name="notify_new_streams">Livestream notifications</string> |
There was a problem hiding this comment.
It has nothing to do with livestreams, it just notifies for new uploaded videos by channels the user subscribed to
app/src/main/res/values/strings.xml
Outdated
| <string name="checking_frequency">Checking frequency</string> | ||
| <string name="new_streams_count">%1$s new streams are available</string> | ||
| <string name="new_streams_by">New streams by %1$s …</string> | ||
| <string name="notify_new_streams">Livestream notifications</string> |
There was a problem hiding this comment.
| <string name="notify_new_streams">Livestream notifications</string> | |
| <string name="notify_new_streams">Subscription news</string> |
Co-authored-by: Allan Nordhøy <epost@anotheragency.no>
List for future installments:
SponsorBlock options are a bit too long.
piped_summary" and similar aren't used?
Also,
Some options could be grayed out/removed when not active/available.