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

Translation capability for branch 0_15 #4274

Draft
wants to merge 11 commits into
base: 0_15
Choose a base branch
from

Conversation

Sojourneer
Copy link

Translation capability for WLED. Google translated language files provided for JA, FR, DE. Has been tested on an ESP8266.

The coverage is high but not perfect; manual addition of text is required. A more robust solution using a screen scraper on a running WLED instance is probably a better way to go.

From the introduction to the I18N/tools (see wled00/I18N/tools/I18N.md for the details):

WLED Localization

Localization for WLED provides the ability to switch between display languages dynamically from the UI settings page. This setting is local to the browser, and is effective immediately (no Save or page refresh is required).

The rest of this document explains the process for providing this capability.

In summary, the UISimulator Web Server and a couple of python scripts are used to extract, translate, and test the translations on a static simulation of WLED. The translation files are then committed to Github, from which the L12N script included on each HTML page will access and apply the translation.


@softhack007 softhack007 added enhancement discussion python Pull requests that update Python code javascript Pull requests that update Javascript code labels Nov 11, 2024
@softhack007
Copy link
Collaborator

softhack007 commented Nov 11, 2024

@Sojourneer please clean up you PR - it seems there are lots of copied files and "cached whatever" that look like backups or intermediate files.

I'll mark your PR as draft for the time being.

@softhack007 softhack007 marked this pull request as draft November 11, 2024 15:24
@@ -0,0 +1,2 @@
cd wled00
sudo python3 UISimulator/main.py "$@"
Copy link
Collaborator

Choose a reason for hiding this comment

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

sudo should not be needed in a public repository.

Copy link
Author

Choose a reason for hiding this comment

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

I don't know what that means. For example, is it permissible in documentation in a public repository?

@softhack007
Copy link
Collaborator

The rest of this document explains the process for providing this capability.

Where is this document?

@Sojourneer
Copy link
Author

@Sojourneer please clean up you PR - it seems there are lots of copied files and "cached whatever" that look like backups or intermediate files.

I'll mark your PR as draft for the time being.

There are no files that are purely generated from something else.

@Sojourneer
Copy link
Author

From the introduction to the I18N/tools (see wled00/I18N/tools/I18N.md for the details):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement javascript Pull requests that update Javascript code python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants