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

Minor edits #32

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

Minor edits #32

wants to merge 5 commits into from

Conversation

n-berenice
Copy link

Minor changes to grammar. I might be able to edit the smartphone screenshots as well but getting the fonts and alignment right is proving to be difficult as they're in .png instead of layered XCF.

& doesn't exists in Spanish
Adjectives are mostly postpositive in the Spanish syntax.
The use of "el" would imply Hiragana and Katakana are one and the same alphabet when they're not, so "los" is better when talking about two or more different things.
The "Ni" conjunction is better suited for the sentence.
i18n/es.json refers to Hiragana and Katakana as syllabaries, this is correct as they're not alphabets.
Copy link
Owner

@aeri aeri left a comment

Choose a reason for hiding this comment

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

Ironically the Spanish translation is not as it should be, thanks for the corrections.

Although the character "ampersand" is a non-alphabetizable symbol in the Spanish alphabet, its use in the name of the application is purely for aesthetic purposes.

Do you propose also translate the images?

@sakci
Copy link
Contributor

sakci commented Jan 4, 2022

Do you propose also translate the images?

I think it would be better if we gave you the translation of the images and you edit it yourself, since:

  • We don't know the name of the font you're using.
  • Phone screenshots should also be replaced with the localized ones and it'd be cumbersome for us to do it. First of all, we don't have the image template. Second, we'd have to compile the app ourselves just for the screenshots. You'll compile a new version for the update anyway so it'd be better if you handle it.

@n-berenice
Copy link
Author

Agree with sakci, I tried to translate the images but they're too complex to edit the easy way, the backgrounds aren't solid meaning if I have to cut something I'll end up with a white rectangle with no background.

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

Successfully merging this pull request may close these issues.

3 participants