Skip to content

[release/v7.6.7] Prevent localization of WPF key gesture strings - #28097

Merged
Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/28020-a049fa38e
Sep 24, 2026
Merged

Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/28020-a049fa38e

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

Backport of #28020 to release/v7.6.7

Triggered by Dongbo Wang (@daxian-dbw) on behalf of Dongbo Wang (@daxian-dbw)

Original CL Label: CL-General

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Customer Impact

  • Customer reported
  • Found internally

Fixes #27994 where WPF key gesture tokens used by Out-GridView and Show-Command were being translated in localized resources, causing those commands to fail in non-English locales. This PR preserves the key gesture tokens and protects API identifiers embedded in invariant resource messages so they are not localized.

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

Regression introduced when the WPF key gesture strings were exposed for localization; they were translated in non-English locales, breaking Out-GridView and Show-Command. Reported in #27994.

Testing

Verified that Out-GridView and Show-Command key gestures function correctly under non-English locales once the key gesture tokens are preserved from localization. Backport verified by confirming the cherry-pick applied cleanly with no conflicts on release/v7.6.7.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Low risk: marks specific resource strings as non-localizable/protected; the localized .resx files are auto-updated by the Localization Pipeline. Narrow, well-scoped fix for a reported customer bug.

Copilot AI lite review requested due to automatic review settings September 24, 2026 21:29
@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Sep 24, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified that would block approval.

Review effort: Lite
Findings: None

What changed in this PR

Adds localization metadata to preserve WPF key gesture tokens and invariant API identifiers, fixing localized Out-GridView and Show-Command failures.

Changes:

  • Protects eight zoom shortcut strings from translation.
  • Protects Columns, Items, and View identifiers in invariant messages.
File Description
src/​Microsoft.Management.UI.Internal/​resources/​public.UICultureResources.resx Updated as part of this pull request.
src/​Microsoft.Management.UI.Internal/​resources/​public.InvariantResources.resx Updated as part of this pull request.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 68964a3 into PowerShell:release/v7.6.7 Sep 24, 2026
35 of 36 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.6.7/28020-a049fa38e branch September 24, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants