Skip to content
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

App: Added more strings in resources #650

Merged
merged 13 commits into from Jan 15, 2024

Conversation

afischerdev
Copy link
Collaborator

Moved more hardcoded strings to resource.
Initial pull request was #645
Please note: this contains no translations.

estimated time = %6$s</string>
<string name="msg_read_wpt_error">Error reading waypoints</string>
<string name="msg_no_wpt">coordinate source does not contain any waypoints!</string>
<string name="msg_to_much_wpts">"coordinate source contains too much waypoints: %1$d (please use from/to/via names)</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

The " in "coordinateshould not be needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are right

@devemux86
Copy link
Contributor

@afischerdev very nice, there are a lot of strings!

Please use String.format(...) with the positional arguments as seen here.

@afischerdev
Copy link
Collaborator Author

@devemux86

Please use String.format(...)

Thanks for the suggestion. I started that way but getString with parameter was working as well.
Why do you think this is a better way?

@devemux86
Copy link
Contributor

@afischerdev

Although the other method is Android specific, Android shows warnings when we build a release apk.
So I had to use in the other pull request the known Java method which passes all tests without issues.

@afischerdev
Copy link
Collaborator Author

@devemux86
Ok, I'll have an eye on that.

@devemux86
Copy link
Contributor

devemux86 commented Dec 13, 2023

@afischerdev there are more texts we need to move to strings.xml:

<string name="summary_version">The base version for tiles has changed. What to do?</string>
<string name="action_version1">Continue with current download, delete other old data</string>
<string name="action_version2">Select all for download and start</string>
<string name="action_version3">Cancel now, complete on an other day</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be "another" day?

<string name="summary_version_diff">The base version for some tiles is different. What to do?</string>
<string name="action_version_diff1">Download all different tiles</string>
<string name="action_version_diff2">Drop all different tiles</string>
<string name="action_version_diff3">Cancel now, complete on an other day</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be "another" day?

<string name="msg_download_failed">Download failed</string>
<string name="msg_download_cancel">Download cancelled</string>
<string name="msg_download_succeed">Download succeeded</string>
<string name="msg_download_started">"Downloading .. "</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

The quotes here are not needed?

@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="utf-8"?>
<resources xmlns:tools="http://schemas.android.com/tools">
<plurals name="numberOfSegments" tools:ignore="MissingQuantity">
<item quantity="one">%d τμήμα</item>
<item quantity="one">%d τμήμα</item>compare cl
Copy link
Contributor

Choose a reason for hiding this comment

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

The "compare cl" is error here, we must remove it.

@afischerdev afischerdev merged commit 5979df1 into abrensch:master Jan 15, 2024
1 check passed
@afischerdev afischerdev deleted the app-trans branch January 16, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants