-
Notifications
You must be signed in to change notification settings - Fork 7.2k
[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
Conversation
This comment has been minimized.
This comment has been minimized.
794290a
to
07c5c7e
Compare
This comment has been minimized.
This comment has been minimized.
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this 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
src/modules/cmdpal/ext/Microsoft.CmdPal.Ext.Apps/Commands/CopyPathCommand.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this 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.
This comment has been minimized.
496c571
to
cd2c832
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
## 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.  ## Validation Steps Performed Manually tested copying for UWPs and Win32 programs
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.
Validation Steps Performed
Manually tested copying for UWPs and Win32 programs