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

packages: title localization fix #12062

Conversation

FernandoAscencio
Copy link
Contributor

@FernandoAscencio FernandoAscencio commented Jan 11, 2023

What it does

Closes #11954 (comment).

The commit fixes localization bugs found in the titles of windows.

While in the process of solving this issue, a check on other title variables found several other windows with similar localization issues. In total six issues should be fixed.

How to test

To test if the initial issue was solved:

  1. start the application
  2. open the debug-view (ex: view > debug)
  3. from the debug-view in the watch section click add expression (+ icon)
  4. a dialog will appear with the title edit watch expression
  5. use the command configure display language (using F1) and install a new language
  6. restart the application when prompted
  7. repeat the steps 1-4 and confirm that the title of the dialog is localized

Review checklist

Reminder for reviewers

Signed-off-by: FernandoAscencio fernando.ascencio.cama@ericsson.com

@FernandoAscencio FernandoAscencio force-pushed the fa/dialogueTitleLocalization branch from aa0623e to 413a2d6 Compare January 12, 2023 15:01
@vince-fugnitto vince-fugnitto added the localization issues related to localization/internalization/nls label Jan 12, 2023
The commit fixes six localization bugs related titles of windows and messages.

Signed-off-by: FernandoAscencio <fernando.ascencio.cama@ericsson.com>
@FernandoAscencio FernandoAscencio force-pushed the fa/dialogueTitleLocalization branch from 413a2d6 to 59892d5 Compare January 12, 2023 16:16
Corrections to the previous localization title fix as suggested in the Reviews

Signed-off-by: FernandoAscencio <fernando.ascencio.cama@ericsson.com>
Another review correction

Signed-off-by: FernandoAscencio <fernando.ascencio.cama@ericsson.com>
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

The changes look good to me 👍

@vince-fugnitto vince-fugnitto requested a review from msujew January 12, 2023 17:57
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Looks good to me as well 👍

FernandoAscencio and others added 3 commits January 16, 2023 09:01
Co-authored-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
The commit fixes six localization bugs related titles of windows and messages.

Signed-off-by: FernandoAscencio <fernando.ascencio.cama@ericsson.com>
@vince-fugnitto vince-fugnitto merged commit c6cb7be into eclipse-theia:master Jan 16, 2023
@FernandoAscencio FernandoAscencio deleted the fa/dialogueTitleLocalization branch January 16, 2023 16:16
@paul-marechal paul-marechal added this to the 1.34.0 milestone Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localization issues related to localization/internalization/nls
Projects
None yet
Development

Successfully merging this pull request may close these issues.

debug: add 'edit watch expression' localization
4 participants