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

Comprehensive Update for Traditional Chinese #8008

Closed
wants to merge 20 commits into from
Closed

Conversation

shuwn
Copy link
Contributor

@shuwn shuwn commented Jan 11, 2025

Description

This PR provides a comprehensive update to the Traditional Chinese localization of the OrcaSlicer project. It addresses inconsistencies and outdated translations.

Updated localization files: Refined translations in OrcaSlicer_zh_TW.po to align with modern terminology and contextually accurate phrasing.

Issues Addressed:

  • Fixes inconsistencies in Traditional Chinese translations.
  • Aligns localization with the latest feature updates introduced in OrcaSlicer.

Screenshots/Recordings/Graphs

473145492_10227772669088235_7631067962325038530_n

Tests

  1. Reviewed the added Traditional Chinese translations to ensure accuracy and consistency.
  2. Tested the application to confirm no unintended side effects in the Ul or functionality.

shuwn and others added 20 commits January 7, 2025 14:44
1.Fix: wrong extruder_counts by Switching between multi-extruder printers using tabs 2. unneccessary unsaved dialog
* fix: tray_exist_bits != tray_exist_bits comparison

* fix: title == GetTitle() == title comparison

* fix: possibly dangling reference to a temporary
```
OrcaSlicer/src/libslic3r/calib.cpp:456:17: warning: possibly dangling reference to a temporary [-Wdangling-reference]
  456 |     const auto &w         = bed_ext.size().x();
OrcaSlicer/src/libslic3r/calib.cpp:456:45: note: the temporary was destroyed at the end of the full expression ‘((Eigen::DenseCoeffsBase<Eigen::Matrix<double, 2, 1, 2>, 1>*)(& Slic3r::BoundingBoxBase<PointClass>::size() const [with PointClass = Eigen::Matrix<double, 2, 1, 2>]()))->Eigen::DenseCoeffsBase<Eigen::Matrix<double, 2, 1, 2>, 1>::x()’
  456 |     const auto &w         = bed_ext.size().x();
```

* fix: mixup of | and ||
in this case I think it actually does not change the semantics it just means that both comparison have to be evaluated.

* fix: multi-character character constants need "

OrcaSlicer/src/slic3r/GUI/MediaPlayCtrl.cpp:392: warning: multi-character character constant [-Wmultichar]
  392 |     if (auto n = tunnel.find_first_of('/_'); n != std::string::npos)
OrcaSlicer/src/slic3r/GUI/MediaPlayCtrl.cpp: In member function ‘void Slic3r::GUI::MediaPlayCtrl::Stop(const wxString&)’:
OrcaSlicer/src/slic3r/GUI/MediaPlayCtrl.cpp:392: warning: overflow in conversion from ‘int’ to ‘char’ changes value from ‘12127’ to ‘95’ [-Woverflow]

* fix: missing paranthesis - skips null check

* NFC: Remove this check it can never be false

* NFC: fix warning: statement has no effect
* Update OrcaSlicer.pot

Made sentence a bit clearer.

* Update PrintConfig.cpp

Made sentence a bit clearer.

* Update OrcaSlicer.pot

corrected previous expression

* Update PrintConfig.cpp

corrected previous expression
@shuwn shuwn closed this Jan 14, 2025
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.

7 participants