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

Added the Romanian language and an experimental Romanian translation #80

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

stflavian
Copy link
Contributor

Following the examples of the previous entries, I added the option to use Romanian words for typing. For now, I used the recommended list of words from Monkeytype's language directory, but it would be a good idea to use a different list in the future. The Romanian language has 5 special characters (ă, â, î, ș, ț) which are often replaced by their standard Latin counterparts (a, i, s, t) in casually written text. As such, it would be nice to have a simple list without special characters, and an advanced list for the full range of characters. Apart from that, I have changed the quotation marks for Romanian, as it uses an identical rule to Bulgarian.

I also translated the app in Romanian, but since I never use the Romanian interface I am not sure if the phrasing used is in line with Gnome conventions. I tested it and it looks fine, so albeit not perfect, I think it is a good start. I have noticed that the About page is not translated, but I am not sure if that is intended or not.

@stflavian stflavian marked this pull request as ready for review November 3, 2024 21:43
@bragefuglseth
Copy link
Owner

bragefuglseth commented Nov 5, 2024

Hi, thanks a lot for the contribution! This looks good to me.

I have noticed that the About page is not translated, but I am not sure if that is intended or not.

The about page and its translations comes from libadwaita, which can be translated on GNOME's centralized translation system. The text on the "What's New" page will always be in English since changelogs usually don't get translated.

@bragefuglseth bragefuglseth merged commit 1040853 into bragefuglseth:main Nov 5, 2024
1 check passed
@stflavian
Copy link
Contributor Author

I am glad I could help! I will be closing the issue I have created as well.

@stflavian stflavian mentioned this pull request Nov 5, 2024
2 tasks
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.

2 participants