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

Localization improvements #156

Closed
FireEmerald opened this issue Feb 26, 2017 · 9 comments · Fixed by #157
Closed

Localization improvements #156

FireEmerald opened this issue Feb 26, 2017 · 9 comments · Fixed by #157
Assignees
Labels
Enhancement Issue which are a enhancement

Comments

@FireEmerald
Copy link
Contributor

I was looking inside your code for the localization and thought it would be more or less easy possible to add another language for SoundSwitch. If this is no problem for you @Belphemur i will add German as 3rd one.

While adding the new *.de.resx files, i'm going to change some names of various controls to get a more consistent naming. This will provide in the future a easier way to implement even more languages.

I'm sure there are also some untranslated strings - like those from the new update settings. I will also add these to the existing resource files as well as some typo fixes ;)

When it's done, I will create a pull request. (next few days)

@Belphemur
Copy link
Owner

Hey

Sure that'd be great. There is couple of issues with the Designer not taking the resource strings (and always a pain in the ass to remove all those "text" by the resource key).

Please don't change the current existing key. To add german, use Zeta Resource Editor: https://www.zeta-resource-editor.com/index.html

It's free and will by default copy the english string in german.

@FireEmerald
Copy link
Contributor Author

FireEmerald commented Feb 26, 2017

Which "existing key"s do you mean shouldn't be changed exactly?

These?
2017-02-26 22_56_40-soundswitch - microsoft visual studio

@Belphemur
Copy link
Owner

Yup exactly, those keys.

I'm quite used to them by now.

@Belphemur Belphemur added the Enhancement Issue which are a enhancement label Feb 26, 2017
@FireEmerald
Copy link
Contributor Author

I'm going to keep (most) of the key names but some are not named consistent. Theres CamelCase other are lowerCamelCase as well as 'cryptical shorted names'.

@Belphemur
Copy link
Owner

Belphemur commented Feb 27, 2017 via email

@FireEmerald
Copy link
Contributor Author

Setting all *.Text = ""; properties inside the *.Designer.cs files is a horror, because the *.Designer.cs files are overwritten each time you change anything inside the form, like resizing a control and so on.

Wouldn't it a way better just to set all Text properties in the constructor of all forms? Seems like the best solution for me.

Ok, we will have to set the Text twice, inside the designer of the form and in the resx files but it's a way easier to sync these files then going through the *.Designer.cs files each time and replace all texts...

@Belphemur
Copy link
Owner

In that, case better create a method in the form that would take care of that (because let's be honest, the constructors are quite full already).

Of course, that method will be called in the constructor.
What I don't understand with the Designer, is that before I was able to take the strings directly from the Properties file ... then after one of the VS update, it stopped doing that.

@FireEmerald
Copy link
Contributor Author

I already replaced all strings from the designer and it's a waaay better now ;) As you already proposed i moved the localization in it's own method(s). There are many changes - it would be better to not push any other commits until i set up the pull request - to avoid merge conflicts.

I think it should be done tomorrow.

@Belphemur
Copy link
Owner

Belphemur commented Feb 28, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Issue which are a enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants