Skip to content
This repository has been archived by the owner on Jan 1, 2024. It is now read-only.

Added a Norwegian Bokmål translation #718

Merged
merged 20 commits into from
Jun 23, 2019
Merged

Conversation

DandelionSprout
Copy link

I've attempted to create a Norwegian translation of DS4Windows this afternoon.

It wasn't easy for me to figure out which files I should copy/create, which strings I should translate, and so on, so I can only assume that I did a decent job with it.

@Ryochan7
Copy link
Owner

I don't know why but app performance is worse with these changes applied. The resource files don't seem to be formatted like the other lang resource files either. In its current state, I cannot accept this pull request. Sorry.

@DandelionSprout
Copy link
Author

DandelionSprout commented Jun 10, 2019

I'll take an unqualified guess that the performance tank was due to me not removing any of the coordinate/meta strings in the translations.

While I consider myself to be pretty good at translating programs, .resx is pretty much witchcraft to me. You can see if it goes any better now that I've tried to remove non-translatable strings from the files; if not, then I'd have to throw in the towel on this one.

@hydraheim
Copy link

hydraheim commented Jun 13, 2019

There are phrasings here that actively translate the originals word-by-word, a common sign for translations that require more work.

(That said, it looks like the original keys should have a pass or two themselves, as they're not up to par in several locations. That's however outside the scope of this observation.)

"<type profile name here>" -> "Skriv inn navnet til den nye profilen her"

Instead of "Skriv inn profilnavn"

Furthermore there are translations that contains spelling/grammatic errors.

"Trinn 3: koble til DualShock 4-kontrolleren
For å sette opp med kabling/USB:
Bare plugg en MicroUSB-kabel til PCen og DualShock 4"

Instead of

"Trinn 3: koble til DualShock 4-kontrolleren
For å sette opp en kablet/USB-tilkobling: Bare plugg en MicroUSB-kabel mellom PC-en din og DualShock 4-en".

This is just a very small excerpt looking at some of the changes. One would believe there are others. In any case, thanks for your efforts.

@DandelionSprout
Copy link
Author

As much as I like to think of myself as being a skilled translator, I must admit that you're correct about both examples.

I translated the initial 5 files in somewhat of a hurry, as I at the time was less than certain about which files/strings I had to translate, leading me to write on my keyboard on autopilot to see how many strings I actually had to translate or not.

Now that I've removed superfluous lines anyway, that should give me enough of an overview to do a bit of an overhaul of the largest files I did.

@Ryochan7
Copy link
Owner

Can you commit the changes from DS4Windows/DS4Windows.csproj?

@DandelionSprout
Copy link
Author

I am currently partway through a week-long vacation, so I'd be dependent on access to a stable Wi-Fi connection where I know that I can edit that file's language codes (presuming that it's what you intend for me to do) for up to half an hour on my somewhat small Lenovo phone. The very earliest I can do so tonight would be ~17:00 UTC.

I caught a cold on Monday afternoon and had to wait until now to do it.
@DandelionSprout
Copy link
Author

Okay, so I've now commited the change you had made in 3473fe1#diff-7497ddfad9fb3defb21dd4485ea80e78, and added some Norwegian Bokmål language codes to the places I've thought they had to be added to.

@Ryochan7
Copy link
Owner

I pulled that latest changes and I encountered the same issues. The main problem is that the new translation files were not added to the DS4Windows.csproj file. After that change was made, the files show up in the solution like the other translation files and app performance is on point. With the extra change, this is ready to merge.

@Ryochan7 Ryochan7 merged commit cca00d8 into Ryochan7:jay Jun 23, 2019
@Ryochan7
Copy link
Owner

Thank you for the updated translations. The changes have been merged.

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

Successfully merging this pull request may close these issues.

3 participants