Skip to content

Fix xsl0101 toggle description in TZM locale (#3133)#7127

Open
niuniu319 wants to merge 1 commit into
FAForever:developfrom
niuniu319:fix/xsl0101-toggle-description
Open

Fix xsl0101 toggle description in TZM locale (#3133)#7127
niuniu319 wants to merge 1 commit into
FAForever:developfrom
niuniu319:fix/xsl0101-toggle-description

Conversation

@niuniu319
Copy link
Copy Markdown

@niuniu319 niuniu319 commented May 22, 2026

Fix #3133

The TZM (French) locale for xsl0101_toggle_description incorrectly included extra text "Toggles Motion-Cloak ability" that is not part of the original English string.

Before: Toggles selection priority\nToggles Motion-Cloak ability
After: Toggles selection priority

This matches all other locales (US, FR, PL, DE, ES, etc.) which correctly use "Toggles selection priority" only.

Closes #3133

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

📝 Walkthrough

Walkthrough

A single localization string in the Turkish (TZM) strings database is corrected: the toggle description for xsl0101_toggle_description no longer includes the redundant "Motion-Cloak" ability reference, clarifying the tooltip to only mention selection priority toggling.

Changes

Selen Stealth Tooltip Fix

Layer / File(s) Summary
Toggle description string correction
loc/TZM/strings_db.lua
The xsl0101_toggle_description string is shortened by removing the second line that incorrectly referenced toggling "Motion-Cloak", leaving only the accurate "toggles selection priority" description.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

A stealth-clad Selen stands tall,
Her tooltip now speaks truth to all—
No Motion-Cloak confusion here,
Just "selection priority" crystal clear! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: fixing the xsl0101 toggle description in the TZM locale, which is exactly what the code change accomplishes.
Linked Issues check ✅ Passed The PR successfully addresses issue #3133 by removing the extraneous 'Toggles Motion-Cloak ability' phrase from the TZM locale string, aligning it with other locales.
Out of Scope Changes check ✅ Passed The changes are limited to a single localization string correction in TZM locale, directly addressing the linked issue with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description clearly identifies the issue, provides before/after comparison, and references the linked issue #3133. It explains the problem and solution with context about other locales.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Wrong tooltip for Selen stealth

1 participant