Skip to content

Localize Tutorial Tool and forward locale to embedded editors - #11626

Open
Christian BR (ChrisesDK) wants to merge 2 commits into
microsoft:masterfrom
ChrisesDK:tutorial-tool-localization-pr
Open

Christian BR (ChrisesDK) wants to merge 2 commits into
microsoft:masterfrom
ChrisesDK:tutorial-tool-localization-pr

Conversation

@ChrisesDK

Copy link
Copy Markdown

Summary

Adds generic localization support to the current React-based MakeCode Tutorial Tool and forwards the active locale to embedded MakeCode editors.

Changes

  • initializes Tutorial Tool localization using the existing PXT localization runtime
  • adds Tutorial Tool strings to the normal PXT localization extraction and publication pipeline
  • adds TranslationsKind.TutorialTool runtime support
  • localizes 9 user-facing React Tutorial Tool strings
  • forwards lang / forcelang to embedded MakeCode editors while preserving existing locale parameters
  • adds locale forwarding support to the legacy Tutorial Tool iframe path without claiming to localize the legacy shell
  • adds focused tests for locale forwarding, default-English behavior, extraction, and runtime catalog loading

Validation

  • gulp tutorialtoolTest - 11 passing
  • Tutorial Tool extraction - 36 files scanned, 9 keys extracted
  • gulp pxtlib - passed
  • gulp testpxteditor - 66 passing
  • git diff --check - passed

The full React build was not run locally because the local Tutorial Tool dependency installation does not contain react-app-rewired. The standalone Tutorial Tool typecheck is also blocked by the existing pxtarget.d.ts types configuration issue; neither issue is introduced by this patch.

Scope

This PR is generic and does not contain Danish-specific translations.

Out of scope:

@ChrisesDK
Christian BR (ChrisesDK) force-pushed the tutorial-tool-localization-pr branch 2 times, most recently from 342016a to 56dc276 Compare October 22, 2026 09:42
@ChrisesDK
Christian BR (ChrisesDK) force-pushed the tutorial-tool-localization-pr branch from 56dc276 to a9fac7d Compare October 26, 2026 10:31
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