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

Norwegian Localisation #679

Merged
merged 15 commits into from
Feb 17, 2021
Merged

Norwegian Localisation #679

merged 15 commits into from
Feb 17, 2021

Conversation

vkbo
Copy link
Owner

@vkbo vkbo commented Feb 17, 2021

This PR adds Norwegian localisation, and also adds a number of newly discovered missing translation locations.

@vkbo vkbo added this to the Release 1.3 Beta 1 milestone Feb 17, 2021
@vkbo
Copy link
Owner Author

vkbo commented Feb 17, 2021

I keep finding strings that aren't translated, although not as often as before. I'm adding them to this PR, so after this is merged, you should probably have a quick update of the Portuguese .ts file too, @bkmeneguello.

@bkmeneguello
Copy link
Contributor

Those missing translations are due some merges I had to do. Some of them I remember have translated 😬

@vkbo
Copy link
Owner Author

vkbo commented Feb 17, 2021

Ok, I'm done and will merge this now. It is a lot of work translating 920 strings!

There seems to be about 20 or so missing in Portuguese now. I recovered a few that I changed in the code. Most are new strings that were missing.

I don't understand Portuguese enough to check, but I made this mistake in Norwegian, so just note that the word "Characters" both refers to the counting of text characters (letters) in the text files and other times to characters (people) in the story. In English this is the same word, but not in Norwegian. Don't know about Portuguese.

@vkbo vkbo merged commit e60cc7b into dev Feb 17, 2021
@vkbo vkbo deleted the norwegian_locale branch February 17, 2021 22:25
@vkbo vkbo linked an issue Feb 17, 2021 that may be closed by this pull request
@bkmeneguello
Copy link
Contributor

bkmeneguello commented Feb 17, 2021 via email

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.

Translations of novelWriter
2 participants