-
-
Notifications
You must be signed in to change notification settings - Fork 125
Add basic About page for app #377
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
base: master
Are you sure you want to change the base?
Conversation
app/src/main/res/values/strings.xml
Outdated
| <string name="about_github">GitHub</string> | ||
| <string name="about_legal">Legal</string> | ||
| <string name="about_license">License</string> | ||
| <string name="about_license_gpl">GNU General Public License v3.0</string> |
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.
Should this be marked non-translatable?
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.
Ahh, fair enough. Haven't messed with the donottranslate.xml file yet, I can move that over.
Edit: I guess just mark with translatable="false" ?
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.
Marked about_github and about_license_gpl as non-translatable. I'm not sure how other apps handle Legal and Privacy Policy (like if it needs to/should stay in English?)
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.
Looks good, I think that is enough, my understanding is only brand or specific names should not be translated.
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.
I feel like this page could be simplified by creating a helper function for each category. Might be better?
Added a basic About page to the app. Privacy policy section is just a copy-paste of PRIVACY.md. Not sure how that should be handled going forward but the basic framework setup.
Resolves #24, #281