Skip to content

Optimize laptop-mode-tools to support internationalization#170

Open
mrdengbo wants to merge 3 commits intorickysarraf:lmt-upstreamfrom
mrdengbo:lmt-upstream
Open

Optimize laptop-mode-tools to support internationalization#170
mrdengbo wants to merge 3 commits intorickysarraf:lmt-upstreamfrom
mrdengbo:lmt-upstream

Conversation

@mrdengbo
Copy link
Copy Markdown
Contributor

Support laptop-mode-tools gui title display internationalization
Fix the problem that the root privileges prompt does not display the corresponding user when the system is in the Chinese environment
Added instructions for developers of internationalized languages

Support laptop-mode-tools gui title display internationalization
Fix the problem that the root privileges prompt does not display the corresponding user when the system is in the Chinese environment
Added instructions for developers of internationalized languages
@mrdengbo
Copy link
Copy Markdown
Contributor Author

In this submission, I added a description of the language internationalization of laptop mode tools and optimized some minor issues

<name>MainWidget</name>
<message>
<location filename="../gui/lmt.py" line="55"/>
<location filename="../../../../gui/lmt.py" line="53"/>
Copy link
Copy Markdown
Owner

@rickysarraf rickysarraf Sep 6, 2021

Choose a reason for hiding this comment

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

Is this path correct ?

I assume the CWDIR would be usr/share/laptop-mode-tools/locale/ and in your proposed patch, if you traverse ../../../../ you are essentially chdir()ing to /

Can you please enlighten on this ?

Support laptop-mode-tools gui title display internationalization
Fix the problem that the root privileges prompt does not display the corresponding user when the system is in the Chinese environment
Added instructions for developers of internationalized languages
@mrdengbo
Copy link
Copy Markdown
Contributor Author

I added a script under the / directory to generate the.ts file through the script, so that the path of the filename is correct

@rickysarraf rickysarraf self-assigned this Apr 12, 2022
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.

3 participants