-
Notifications
You must be signed in to change notification settings - Fork 60
Develop ESP translation #750
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
base: develop
Are you sure you want to change the base?
Conversation
Fix menu for translations
Fix few docs issues with es
Added device es imgs
|
Hi @westoleaboat , what do you think about merging this now to bring in the new theme and other updates? That way, everyone can already see what’s been done in Spanish - and if they notice anything missing or still in English, it might encourage more contributions. Otherwise, I’m afraid this PR might take quite a while... |
|
hi @tadeubas yes we should do it. |
|
Opened (another 😄) PR to you: westoleaboat#4 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #750 +/- ##
========================================
Coverage 97.37% 97.37%
========================================
Files 83 83
Lines 10525 10534 +9
========================================
+ Hits 10249 10258 +9
Misses 276 276 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Updated image references to use absolute paths instead of relative links and more!
New docs spanish
|
Thank you for you hard work so far guys. Docs are looking great. |
qlrd
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits about src=" /krux/img attribute on html stuffs. This creates unecessary diff and is out of the scope of the PR change the src attribute. Please revert it to relative path.
Also, I think this PR is still in Draft process, so could be better to change its visibility.
| ### Standard Mode | ||
| <img src="../../../img/maixpy_m5stickv/standard-qr-code-250.png" align="right" class="m5stickv"> | ||
| <img src="../../../img/maixpy_amigo/standard-qr-code-300.png" align="right" class="amigo"> | ||
| <img src="/krux/img/maixpy_m5stickv/standard-qr-code-250.png" align="right" class="m5stickv"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the change? IMO this could be potentially be problematic depending on developer's machine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've explained before in our Telegram chat: using those paths would break links in the Spanish version because its URLs include the /es/ prefix. Also the relative paths were messy... if we move a document to a different location would break all the links.
Whether the paths are relative or absolute doesn’t affect how the image is displayed; it will render normally either way, it doesn't depend on developer's machine or our site generate by GitHub actions
| <img src="/krux/img/maixpy_m5stickv/lines-qr-code-250.png" align="right" class="m5stickv"> | ||
| <img src="/krux/img/maixpy_amigo/lines-qr-code-300.png" align="right" class="amigo"> | ||
|
|
||
| If you are good at transcribing things like handwritten text, with this mode one QR code line will be highlighted at a time. Press `ENTER` to highlight the next line. | ||
|
|
||
| <div style="clear: both"></div> | ||
|
|
||
| ### Zoomed Regions Mode | ||
| <img src="../../../img/maixpy_m5stickv/zoomed-qr-code-250.png" align="right" class="m5stickv"> | ||
| <img src="../../../img/maixpy_amigo/zoomed-qr-code-300.png" align="right" class="amigo"> | ||
| <img src="/krux/img/maixpy_m5stickv/zoomed-qr-code-250.png" align="right" class="m5stickv"> | ||
| <img src="/krux/img/maixpy_amigo/zoomed-qr-code-300.png" align="right" class="amigo"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| <img src="/krux/img/maixpy_m5stickv/grided-qr-code-250.png" align="right" class="m5stickv"> | ||
| <img src="/krux/img/maixpy_amigo/grided-qr-code-300.png" align="right" class="amigo"> | ||
|
|
||
| Grids will be added to a standard QR code. In a dark room, if you place a sheet of paper over the device's screen, you'll notice QR code will be visible and it will be possible to copy it directly from above (tracing). Be careful not to damage your screen with pen and markers, use an insulating plastic tape or film to protect the device when using this method. | ||
|
|
||
| <div style="clear: both"></div> | ||
|
|
||
| ### QR Viewer | ||
| <img src="../../../img/maixpy_m5stickv/qr-viewer-250.png" align="right" class="m5stickv"> | ||
| <img src="../../../img/maixpy_amigo/qr-viewer-300.png" align="right" class="amigo"> | ||
| <img src="/krux/img/maixpy_m5stickv/qr-viewer-250.png" align="right" class="m5stickv"> | ||
| <img src="/krux/img/maixpy_amigo/qr-viewer-300.png" align="right" class="amigo"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| <img src="/krux/img/maixpy_m5stickv/load-mnemonic-kef-via-qr-250.png" align="right" class="m5stickv"> | ||
| <img src="/krux/img/maixpy_amigo/load-mnemonic-kef-via-qr-300.png" align="right" class="amigo"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
|
|
||
| <img src="../../../img/mnemonic_xor.png" align="center"> | ||
| <img src="/krux/img/mnemonic_xor.png" align="center"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| @@ -0,0 +1,23 @@ | |||
| <img src="/krux/img/krux-devices.jpg" style="width: 40%; min-width: 260px;" class="align-right"> | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| official desktop application, **Krux-Installer**: | ||
|
|
||
| [<img src="../../../img/badge_github.png" width="186">](https://github.com/selfcustody/krux-installer/releases) | ||
| [<img src="/krux/img/badge_github.png" width="186">](https://github.com/selfcustody/krux-installer/releases) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| @@ -0,0 +1,15 @@ | |||
| Puedes instalar Krux (ambos la versión oficial o beta) en tu dispotivo con chip K210 utilizando nuestra aplicación oficial para computador, **Krux-Installer**: | |||
|
|
|||
| [<img src="/krux/img/badge_github.png" width="186">](https://github.com/selfcustody/krux-installer/releases) | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| After choose `odudex/krux_binaries`, you'll be warned with a message: | ||
|
|
||
| <img width="640" src=" ../../../../img/krux-installer/warn_beta.png" alt="Krux-Installer warning beta version" /> | ||
| <img width="640" src=" /krux/img/krux-installer/warn_beta.png" alt="Krux-Installer warning beta version" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
| At startup, **Krux-Installer** recognize the locale used in your system. If your language isn't supported, it will defaults to `en_US`. | ||
|
|
||
| <img width="640" src=" ../../../../img/krux-installer/locale_menu.png" alt="Krux-Installer locale menu" /> | ||
| <img width="640" src=" /krux/img/krux-installer/locale_menu.png" alt="Krux-Installer locale menu" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too
Images need to have absolute path, otherwise Spanish will fail to get images on pages that doesn't have Spanish translation yet |
|
I would squash some commits that are This PR is hard to review and rebase to test it locally (due to This is one of reasons that I think we should encourage to read a |
Please open a follow-up PR to westoleaboat. I think these requirements are a bit too strict for people submitting their first PRs |
|
Needs a rebase |
What is this PR for?
Translation of docs in Spanish
Changes made to:
Did you build the code and tested on device?
mkdocs serveWhat is the purpose of this pull request?