Skip to content

feat(i18n): update italian language#158

Open
elax46 wants to merge 4 commits into
AprilNEA:masterfrom
elax46:master
Open

feat(i18n): update italian language#158
elax46 wants to merge 4 commits into
AprilNEA:masterfrom
elax46:master

Conversation

@elax46
Copy link
Copy Markdown
Contributor

@elax46 elax46 commented Jun 6, 2026

Update Italian (it) localization entries to openlogi-gui/locales/app.yml. Provides Italian translations for many UI strings across device/pointer/lighting panels, menu bar items, about/help, SmartShift panel, binding popovers, actions and thumb-wheel texts.

elax46 and others added 4 commits June 5, 2026 21:38
Add Italian translations for many UI strings in crates/openlogi-gui/locales/app.yml and register the 'it' locale in crates/openlogi-gui/src/i18n.rs (SUPPORTED list and match_supported). Update tests to exercise the Italian locale and translations. This enables Italian interface text across menus, settings, permissions, pairing flow, DPI/actions and other UI copy.
Co-authored-by: pullfrog[bot] <226033991+pullfrog[bot]@users.noreply.github.com>
Add Italian (it) localization entries to crates/openlogi-gui/locales/app.yml. Provides Italian translations for many UI strings across device/pointer/lighting panels, menu bar items, about/help, SmartShift panel, binding popovers, actions and thumb-wheel texts. Also includes a small tweak to the Traditional Chinese SmartShift message in the same file.
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Jun 6, 2026

Greptile Summary

This PR adds Italian (it) localization entries across ~80 UI strings in app.yml, covering device/pointer/lighting panels, the menu bar, SmartShift panel, binding popovers, actions, and thumb-wheel labels. It also adds the missing zh-TW entries for "Previous Desktop" and "Next Desktop".

  • Italian translations look accurate and complete for all addressed keys; no %{…} interpolation placeholders are missing.
  • The zh-TW value for "Couldn't read SmartShift — click to retry." was accidentally replaced with the translation for "Reading SmartShift settings…", causing the wrong message to appear for Traditional Chinese (Taiwan) users in that error state.
  • An explanatory comment block at the top of the SmartShift section was removed and should be restored.

Confidence Score: 3/5

The Italian translations are correct and complete, but the PR accidentally overwrites an existing zh-TW string with the wrong text, so merging as-is will silently break the Traditional Chinese (Taiwan) error message for SmartShift.

One existing zh-TW entry was overwritten with the wrong string, which will surface as a mistranslated error state for zh-TW users on every SmartShift failure. That regression needs to be corrected before merging.

crates/openlogi-gui/locales/app.yml — specifically the zh-TW entry under "Couldn't read SmartShift — click to retry." at line 750.

Important Files Changed

Filename Overview
crates/openlogi-gui/locales/app.yml Adds Italian (it) translations for ~80 UI strings; accidentally overwrites the zh-TW entry for "Couldn't read SmartShift — click to retry." with the wrong string, and drops a helpful maintainer comment from the SmartShift section header.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["app.yml locale lookup"] --> B{"locale = 'it'?"}
    B -- yes --> C["Return Italian string"]
    B -- no --> D{"locale = 'zh-TW'?"}
    D -- yes --> E["Return zh-TW string"]
    D -- no --> F["Fallback to English key"]
    E --> G["'Couldn't read SmartShift'\nkey now returns wrong string\n(Reading SmartShift settings…)"]
    C --> H["All ~80 new IT strings\nrender correctly"]
Loading

Fix All in Codex Fix All in Claude Code

Reviews (1): Last reviewed commit: "Add Italian translations to app.yml" | Re-trigger Greptile

Comment thread crates/openlogi-gui/locales/app.yml
Comment thread crates/openlogi-gui/locales/app.yml
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.

1 participant