Skip to content

fix(locale): translate leftover English labels - #2795

Merged
EllesmereGaming merged 1 commit into
EllesmereGaming:mainfrom
LoChinAn:fix-untranslated-tooltip-dm-popup-labels
Oct 19, 2026
Merged

EllesmereGaming merged 1 commit into
EllesmereGaming:mainfrom
LoChinAn:fix-untranslated-tooltip-dm-popup-labels

Conversation

@LoChinAn

@LoChinAn LoChinAn commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Problem

On non-English clients these stay English: the M+ Score: / Mount: tooltip lines; Damage Meters Unknown, (N overkill) and the Damage Meter N movers; the Minimap friends tooltip (Whisper, Invite, No friends online, ...and N more); AuraBuff Reminders Unknown; the Raid & Party Frames preview titles and intro popup.

Cause

They go straight to SetText / AddDoubleLine, or are joined with a number before L() sees them.

Fix

Literals are wrapped in L() at the call site; joined strings use Lf(), with color codes kept outside the key. The tooltip duplicate checks now look for the same L() key as the line they guard (as Item Level does), so a translated line is not added twice.

Notes

Testing

Compiles under Lua 5.1; .tools/guardrails.py passes. Not yet loaded in-game.

Checklist

  • N/A - no new settings
  • N/A - no events, hooks or frames added
  • Same cost as before: one catalog lookup per label
  • N/A - no new writes to Blizzard frames
  • Tested in-game on live; no version gates or pre-Midnight APIs added

These labels went straight to SetText / AddDoubleLine, or were joined
with a number before L() saw them, so they stayed English on every
locale. Wrap the literals in L() at the call site and use Lf() for the
joined ones. The tooltip duplicate checks now look for the same L() key
as the line they guard, so a translated line is not added twice.
@EllesmereGaming
EllesmereGaming merged commit 04642f1 into EllesmereGaming:main Oct 19, 2026
1 of 2 checks passed
EllesmereGaming pushed a commit that referenced this pull request Oct 19, 2026
Covers v9.6 (after #2762) through v9.6.2, plus the keys #2795 sends
through L(). Every string that mentions Shifter now uses the page's
own name, and 5 keys no source produces any more are dropped.
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.

2 participants