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

Issue 173 - Credit translator #183

Merged
merged 7 commits into from
Apr 21, 2019
Merged

Conversation

R3ddingt0n
Copy link
Contributor

@R3ddingt0n R3ddingt0n commented Apr 20, 2019

Fixes Issue #173

Well this seems to work.
I am sure there are still many improvements to do, so if there's something I can do, let me know 😉

Example with browser configured in french :

fr

Example with browser configured in german, where the english text is shown :

en

@rugk
Copy link
Owner

rugk commented Apr 21, 2019

Thanks. I've changed the styling a little and fixed some issues in the Localizer lib. Also added the English example and adjusted the text/i18n selectors.

This is how it looks now:
image

BTW, also written the corresponding doc for this at https://github.com/TinyWebEx/common/blob/master/CONTRIBUTING.md#translator-credit-inside-of-add-on

Copy link
Owner

@rugk rugk left a comment

Choose a reason for hiding this comment

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

So still TODO:

  • translate the German text and test it (/me will do)
  • adjust French text/translation and test it (@nicolas-trouin)

@rugk
Copy link
Owner

rugk commented Apr 21, 2019

BTW to (automatically) let issues close when a PR is merged, write "fixes" before the issue number (in your heading here).

@rugk
Copy link
Owner

rugk commented Apr 21, 2019

BTW, after this is merged, and if you also want to do the same PR for the other add-ons (backlinks above this comment), then feel free to do so… 😄

@R3ddingt0n
Copy link
Contributor Author

BTW, after this is merged, and if you also want to do the same PR for the other add-ons (backlinks above this comment), then feel free to do so… 😄

I'll check it out ^^'

@R3ddingt0n
Copy link
Contributor Author

French translation

image

Is that fine ?

I also spotted a spacing problem with the english translation, as shown below :

image

Should I try to solve it ?

@rugk
Copy link
Owner

rugk commented Apr 21, 2019

Likely you just need to run git submodule update to sync the submodules to the new state I have commit them… (bad git does not do this automatically)

(Because that did not happen in my case.)

@R3ddingt0n
Copy link
Contributor Author

Likely you just need to run git submodule update to sync the submodules to the new state I have commit them… (bad git does not do this automatically)

(Because that did not happen in my case.)

Yep that solved the problem. Anything else I can do about the french translation or other ?

@rugk
Copy link
Owner

rugk commented Apr 21, 2019

I can do about the french translation or other ?

I don't know whether it's still the case, but there likely should not be a space before the ! at the end of the sentence.

@rugk rugk merged commit b2cda77 into rugk:master Apr 21, 2019
@R3ddingt0n
Copy link
Contributor Author

I don't know whether it's still the case, but there likely should not be a space before the ! at the end of the sentence.

Well, I think you'd be right for English or German, but in French there has to be a single space before and after "!", "?", ";", and maybe others too.

R3ddingt0n referenced this pull request Apr 22, 2019
Should work/hopefully be okay. /cc @nicolas-trouin
@rugk
Copy link
Owner

rugk commented Apr 25, 2019

BTW, after this is merged, and if you also want to do the same PR for the other add-ons (backlinks above this comment), then feel free to do so… smile

I'll check it out ^^'

BTW, let me know in rugk/awesome-emoji-picker#18 and rugk/mastodon-simplified-federation#25 if you want to solve that. 😄

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.

None yet

2 participants