-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add the menu option to set Translation source #149
Comments
@henrikth93, Hi, does this work? |
Should work now! |
Hey @ThousandYearGrind 👋 Quick note that this issue is now just to add the menu option that selects from a list, so at the end of it we want the menu option that's shown in the issue text and the interface like this in the Figma designs. Actually switching the translation language will be handled by #265 once this issue is finished - i.e. this issue is for the interface and #265 is for the functionality. |
Ahhh, thank you. I was scratching my head over that for a while. |
@andrewtavis How should I approach translations? Can I just run "Translation source language" through google translate or should this be done later by someone else? |
We'll have local translations for this, @ThousandYearGrind :) This issue is just for the menu option to select the base language. |
Hi @andrewtavis, where would be a good place to ask code-specific questions? I've been trying to figure out how to do something for a while. |
@ThousandYearGrind, asking them here would be good, that way it's easy for us to link to files :) |
@andrewtavis So I've added the option for the settings screen using a ClickableItem like the app language setting. Now, I need to set the action that takes place when the user clicks on the setting.
I looked at the app language setting's action and it uses an intent, but I'm not sure how to do this for the translation language setting.
|
Hey @ThousandYearGrind 👋 Firstly, could we get this menu option moved into the settings for each language keyboard, so it wouldn't be on settings, but on your screen it would be in French and English. We made the decision to have it there as there are members of the community who would translate from different languages: say French to Spanish and English to German. From there we won't be having it function like the app's language selection. Clicking that option will take the user to a sub menu with radio options. Maybe you can make the change for the menu option location and then open a PR and we can then assist on making the radio option menu? |
@andrewtavis I've done as you suggested and moved the menu option to the language keyboard. Should I submit a PR now? |
We're basically there, @ThousandYearGrind! Could you refer to where it should be located in the designs for this tab? Pasting an image of its location in its own sub menu below as well: Feel free to open a PR once it's been moved! |
And thanks for your persistence here! |
Terms
Description
Currently the individual language settings doesn't allow the user to set the translation source for each language keyboard settings. The issue aims to add the ability to set translation source language for each language keyboard separately. This could be stored in the shared preferences for now for each language separately similar to how it is being done for period_on_double_tap.
A visual reference of how this option would look is given below.
Contribution
I would love to work on this and am more than happy to help anyone fix this issue :)
The text was updated successfully, but these errors were encountered: