Skip to content
This repository has been archived by the owner on Jul 26, 2022. It is now read-only.

App strings reworked #332

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

comradekingu
Copy link
Contributor

@comradekingu comradekingu commented Jan 11, 2021

Untranslatable string spotted by @NathanBnm https://hosted.weblate.org/translate/transistor/strings/nb_NO/?checksum=9c1d94904dd3ca43 :)

'Day mode' is different from "light mode"?

<string name="toastalert_headphones_unplugged">Headphones unplugged. Playback stopped.</string>
<string name="toastalert_permission_denied">Permission denied:</string>
<string name="toastalert_station_not_found">Unable to find station.</string>
<string name="snackbar_request_storage_access">Transistor needs to be granted read permissions to be able access data from external storage.</string>
<string name="snackbar_request_storage_access">Grant \"Read storage\" permission to access data from external storage.</string>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this just the one permission?

Copy link
Owner

Choose a reason for hiding this comment

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

On the code level this permission is called "READ_EXTERNAL_STORAGE".

<string name="dialog_error_title_fetch_download">Download Issue</string>
<string name="dialog_error_title_fetch_read">Read Issue</string>
<string name="dialog_error_title_fetch_write">Write Issue</string>
<string name="dialog_error_message_fetch_general_external_storage">External storage:</string>
<string name="dialog_error_message_fetch_general_hint_m3u">Hint: Transistor can only process MP3, AAC and Ogg/Opus encoded streams or links ending in .m3u or .pls.</string>
<string name="dialog_error_message_fetch_general_hint_m3u">Hint: Transistor can only process MP3, AAC and Ogg Opus encoded streams or links ending in .m3u or .pls.</string>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Owner

Choose a reason for hiding this comment

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

As I understand the Wikipedia entry the actual data format of such a stream is OPUS and OGG is just a container. If that is the case we could drop "Ogg" completely, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unless they actually are delivered as Ogg container streams?

Copy link
Owner

Choose a reason for hiding this comment

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

When asked to add an OGG file to the collection of radio stations, Transistor strips out the stream address and does not store the container.

@@ -174,20 +174,20 @@
<!-- error dialogs -->
<string name="dialog_error_details_button_default">Show details</string>
<string name="dialog_error_details_default">No details available</string>
<string name="dialog_error_details_write">This station is probably a duplicate or else you might want to check your external storage.</string>
<string name="dialog_error_details_write">Check your external storage if this station isn't a duplicate.</string>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What would one check external storage for?

Copy link
Owner

Choose a reason for hiding this comment

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

There could be two reasons why the write fails: 1. Duplicate File 2. External Storage unmounted (unlikely). The string is shown when a user taps on "Show details". In an error dialog. I think "Station is probably a duplicate." might be good here.

@@ -128,7 +128,7 @@
<!-- app shortcuts -->
<string name="shortcut_last_station_short_label">Last station</string>
<string name="shortcut_last_station_long_label">Play last station</string>
<string name="shortcut_last_station_disabled_message">Shortcut for playback of last station disabled.</string>
<string name="shortcut_last_station_disabled_message">Removed shortcut for playback of last station.</string>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This isn't great

Copy link
Owner

Choose a reason for hiding this comment

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

👍

<!-- Allowed Template -->
<string name="allowed_caller_log">
The caller has a valid certificate, but the wrong package for it.
To allow this caller, add the following to the allowed callers list:\n
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Where is the "Allowed callers" list?

Copy link
Owner

Choose a reason for hiding this comment

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

I removed the whole string, since it causes errors, as you pointed out a while a ago.

See: aa9c7a8

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 removed it. Could be uncommenting it is better. It caused problems with XML checks over at Weblate.

@y20k
Copy link
Owner

y20k commented Jan 12, 2021

'Day mode' is different from "light mode"?

No. That is the same. In the settings screen it is called: "Light mode". That's the easier to understand term, I guess.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants