Skip to content
This repository has been archived by the owner on Dec 26, 2022. It is now read-only.

[WIP] internationalization #33

Closed
wants to merge 2 commits into from
Closed

Conversation

nanls
Copy link

@nanls nanls commented May 20, 2021

I started working on #22. It is my first time working on a browser add-on. Is this what you had in mind ?

Thanks in advance.

@@ -22,19 +22,19 @@
<div class="panel-section panel-section-formElements">
<div class="panel-formElements-item">
<button id="stop" class="expander">
Mettre en pause la surveillance des RDV
<!-- Mettre en pause la surveillance des RDV -->
Copy link
Owner

Choose a reason for hiding this comment

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

Is this syntax supported?

Copy link
Author

Choose a reason for hiding this comment

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

Sorry, I'm not sure how to parse your question. 😕
I just kept the original texts as html comments, which are indeed supported by Firefox: the texts are hidden in the html rendering and added later by the js code. Is this what you asked?

@hschaeidt
Copy link
Collaborator

@nanls thank you for your inspiration. I took the task over, as this PR seemed to stall a little bit and I needed the i18n for the german translations. This feature has been merged with #91.

However, if you still want to contribute the English translations, all the translatable texts are now extracted to the messages.json files in the _locales folder.

https://github.com/dunglas/vaccin.click/blob/main/_locales/fr/messages.json

@dunglas dunglas closed this Dec 11, 2021
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