Skip to content

Annotate locale-engine fork (mark [fork] changes, neutral naming)#390

Open
labrie75 wants to merge 1 commit into
EllesmereGaming:mainfrom
labrie75:labrie75-patch-1
Open

Annotate locale-engine fork (mark [fork] changes, neutral naming)#390
labrie75 wants to merge 1 commit into
EllesmereGaming:mainfrom
labrie75:labrie75-patch-1

Conversation

@labrie75

Copy link
Copy Markdown
Contributor

Annotate the localization fork; restore upstream-identical lines Make this fork's delta vs upstream explicit and minimal:

  • Restore lines we had incidentally changed so the diff shows only intentional changes (separator widths; drop a stray inline comment).
  • Mark every fork change inline:
    • [fork] MODIFIED — Lf() now L()-translates each argument so a label passed as %s renders localized (upstream original kept in a comment).
    • [fork] ADDED — language-neutral blocks not in upstream:
      • reverse-format engine (localize format()/concat strings from the catalog)
      • render-boundary hooks (FontString/Button:SetText, SetFormattedText, tooltip line pools, EUI StaticPopup) with 12.0 secret-value/taint guards - localized tooltip-line parser (Item Upgrade Calculator, etc.)
  • Rename the install flag to a locale-neutral name (_localeRenderHook). The ADDED blocks translate at the render boundary for ANY active locale; only Locales/koKR.lua is Korean-specific.

Annotate the localization fork; restore upstream-identical lines
Make this fork's delta vs upstream explicit and minimal:
- Restore lines we had incidentally changed so the diff shows only
  intentional changes (separator widths; drop a stray inline comment).
- Mark every fork change inline:
  * [fork] MODIFIED — Lf() now L()-translates each argument so a label
    passed as %s renders localized (upstream original kept in a comment).
  * [fork] ADDED — language-neutral blocks not in upstream:
      - reverse-format engine (localize format()/concat strings from the catalog)
      - render-boundary hooks (FontString/Button:SetText, SetFormattedText,
        tooltip line pools, EUI StaticPopup) with 12.0 secret-value/taint guards
      - localized tooltip-line parser (Item Upgrade Calculator, etc.)
- Rename the install flag to a locale-neutral name (_localeRenderHook).
The ADDED blocks translate at the render boundary for ANY active locale;
only Locales/koKR.lua is Korean-specific.
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