[PR #12491] [MERGED] Copy localized strings from ContextMenu into the resource root #29075

Open
opened 2026-01-31 09:32:37 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/12491
Author: @DHowett
Created: 2/14/2022
Status: Merged
Merged: 2/15/2022
Merged by: @undefined

Base: mainHead: dev/duhowett/copyloc


📝 Commits (3)

📊 Changes

6 files changed (+102 additions, -15 deletions)

View changed files

📝 build/pipelines/release.yml (+5 -0)
build/scripts/Copy-ContextMenuResourcesToCascadiaPackage.ps1 (+34 -0)
📝 src/cascadia/CascadiaPackage/Package-Dev.appxmanifest (+5 -5)
📝 src/cascadia/CascadiaPackage/Package-Pre.appxmanifest (+5 -5)
📝 src/cascadia/CascadiaPackage/Package.appxmanifest (+5 -5)
📝 src/cascadia/CascadiaPackage/Resources/en-US/Resources.resw (+48 -0)

📄 Description

We chose to use the "ContextMenu" resource compartment when we
changed the package name to Terminal in #12264 because it was more
broadly localized than the rest of the application.

It appears as though some platform features have trouble with the
"more qualified" resource paths that #12264 required.

To fix this, we will:

  1. Copy all of the ContextMenu localizations into CascadiaPackage's
    resource root
  2. Switch all manifest resource paths to use resources from the package
    root.

Regressed in #12264
Closes #12384
Closes #12406 (tracked in microsoft/powertoys#16118)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/12491 **Author:** [@DHowett](https://github.com/DHowett) **Created:** 2/14/2022 **Status:** ✅ Merged **Merged:** 2/15/2022 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/duhowett/copyloc` --- ### 📝 Commits (3) - [`5e803e8`](https://github.com/microsoft/terminal/commit/5e803e8b87fe70ed2e309bb00f02b2d5a9ecbce7) Try this! - [`838ac51`](https://github.com/microsoft/terminal/commit/838ac51e45e043b5f0d2b7020d4fe83ad029b6a6) Use CascadiaPackage resources in CascPack manifests - [`2811e09`](https://github.com/microsoft/terminal/commit/2811e09906a775639ddfc10961ba58868ea11498) copydown ### 📊 Changes **6 files changed** (+102 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `build/pipelines/release.yml` (+5 -0) ➕ `build/scripts/Copy-ContextMenuResourcesToCascadiaPackage.ps1` (+34 -0) 📝 `src/cascadia/CascadiaPackage/Package-Dev.appxmanifest` (+5 -5) 📝 `src/cascadia/CascadiaPackage/Package-Pre.appxmanifest` (+5 -5) 📝 `src/cascadia/CascadiaPackage/Package.appxmanifest` (+5 -5) 📝 `src/cascadia/CascadiaPackage/Resources/en-US/Resources.resw` (+48 -0) </details> ### 📄 Description We chose to use the "ContextMenu" resource compartment when we changed the package name to Terminal in #12264 because it was more broadly localized than the rest of the application. It appears as though some platform features have trouble with the "more qualified" resource paths that #12264 required. To fix this, we will: 1. Copy all of the ContextMenu localizations into CascadiaPackage's resource root 2. Switch all manifest resource paths to use resources from the package root. Regressed in #12264 Closes #12384 Closes #12406 (tracked in microsoft/powertoys#16118) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:32:37 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29075