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

Simplified translations #445

Merged
merged 13 commits into from
Aug 24, 2024
Merged

Conversation

tadeubas
Copy link
Contributor

Description

Removing unnecessary translations, merging others to simplify the translation process and also to reduce a little the firmware size

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

@@ -179,7 +179,7 @@ def print_tiny_seed(self):
pad_y = 8 # 2mm*8px
self.ctx.display.clear()
self.ctx.display.draw_hcentered_text(
t("Printing") + " ...", self.ctx.display.height() // 2
t("Printing") + "..", self.ctx.display.height() // 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

I like this pattern (if it works in all languages "..")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have been using this 2 dots pattern since first version of Krux!

https://github.com/selfcustody/krux/blob/v22.03.0/i18n/translations/en-US.json

Copy link
Collaborator

Choose a reason for hiding this comment

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

I meant the t("In the act of doing") + ".." vs t("In the act of doing..") pattern.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But Jeff's file (first version of Krux) doesn't distinguish between going to do or doing, see examples below:

Checking receive address %d for match..
Loading..
Shutting down..
Updating bootloader..
Upgrading firmware..
Backing up bootloader..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry! I saw what you mean now... it is the separation of the ponctuation from the word... I think we can't do that. I will change the above

@jdlcdl
Copy link
Collaborator

jdlcdl commented Aug 17, 2024

This pr has:

  • -654 fewer lines
  • +303 replacing lines

Each translator file has:

  • -27 fewer lines
  • +11 replacing lines

green checks

Wow!

Copy link

codecov bot commented Aug 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.21%. Comparing base (c5e0356) to head (71d8296).
Report is 14 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #445      +/-   ##
===========================================
+ Coverage    94.20%   94.21%   +0.01%     
===========================================
  Files           59       59              
  Lines         7264     7245      -19     
===========================================
- Hits          6843     6826      -17     
+ Misses         421      419       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@odudex
Copy link
Member

odudex commented Aug 24, 2024

Thank you!

@odudex odudex merged commit 04e7cca into selfcustody:develop Aug 24, 2024
7 checks passed
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