Skip to content

[CmdPal][Apps]Add copy path command #39984

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 16, 2025

Conversation

PesBandi
Copy link
Contributor

@PesBandi PesBandi commented Jun 10, 2025

Summary of the Pull Request

Adds a Copy path command to the Ctrl+K menu in the All apps extension

PR Checklist

Detailed Description of the Pull Request / Additional comments

Copies the exe full path for standard programs. Copies the package directory for UWPs.
Shows a Copied to clipboard! toast on a successful copy and closes.
Shows a toast when copying fails, keeps CmdPal open.
image

Validation Steps Performed

Manually tested copying for UWPs and Win32 programs

This comment has been minimized.

@PesBandi PesBandi force-pushed the cmdpal-apps-copypath branch from 794290a to 07c5c7e Compare June 10, 2025 19:22

This comment has been minimized.

@PesBandi

This comment was marked as resolved.

@yeelam-gordon yeelam-gordon requested a review from Copilot June 11, 2025 02:01
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new "Copy path" command to the All apps extension enabling users to copy the full executable path for Win32 programs and package directories for UWPs.

  • Added localized strings for the copy commands in Resources.resx
  • Integrated CopyPathCommand in both Win32Program.cs and UWPApplication.cs
  • Implemented the CopyPathCommand with clipboard operations and toast notifications

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File Description
Resources.resx Added localization entries for copy path, copy failure, and success messages
Win32Program.cs Integrated CopyPathCommand for Win32 programs
UWPApplication.cs Integrated CopyPathCommand for UWP applications
CopyPathCommand.cs New command implementation for copying paths with error handling
Files not reviewed (1)
  • src/modules/cmdpal/ext/Microsoft.CmdPal.Ext.Apps/Properties/Resources.Designer.cs: Language not supported

@yeelam-gordon yeelam-gordon requested a review from lei9444 June 11, 2025 02:20
@yeelam-gordon yeelam-gordon added the Product-Command Palette Refers to the Command Palette utility label Jun 11, 2025
Copy link
Contributor

@moooyo moooyo left a comment

Choose a reason for hiding this comment

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

LGTM

This comment has been minimized.

@PesBandi PesBandi force-pushed the cmdpal-apps-copypath branch from 496c571 to cd2c832 Compare June 11, 2025 19:27
@cinnamon-msft cinnamon-msft added this to the PowerToys 0.92 milestone Jun 12, 2025
@yeelam-gordon
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lei9444 lei9444 merged commit 64ad9b5 into microsoft:main Jun 16, 2025
10 checks passed
@PesBandi PesBandi deleted the cmdpal-apps-copypath branch June 16, 2025 05:07
yeelam-gordon pushed a commit that referenced this pull request Jun 20, 2025
## Summary of the Pull Request
Adds a *Copy path* command to the <kbd>Ctrl</kbd>+<kbd>K</kbd> menu in
the All apps extension
## PR Checklist
- [x] **Closes:** #39500 (as mentioned in the discussion, the other
requests will be tracked over in #39501)
- [x] **Communication:** I've discussed this with core contributors
already + has Help wanted
- [x] **Tests:** All pass
- [x] **Localization:** All end user facing strings can be localized
- [x] **Dev docs:** No need
- [x] **New binaries:** None
- [x] **Documentation updated:** No need
## Detailed Description of the Pull Request / Additional comments
Copies the exe full path for standard programs. Copies the package
directory for UWPs.
Shows a *Copied to clipboard!* toast on a successful copy and closes.
Shows a toast when copying fails, keeps CmdPal open.

![image](https://github.com/user-attachments/assets/ad89a583-13c8-475c-973b-b0f5e7556e31)
## Validation Steps Performed
Manually tested copying for UWPs and Win32 programs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In for .92 Product-Command Palette Refers to the Command Palette utility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants