Skip to content
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

Add action IDs to the color selection commands #17821

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

PankajBhojwani
Copy link
Contributor

Summary of the Pull Request

Add action IDs to the default commands for color selection

Validation Steps Performed

Color selection commands now show up in the command palette

PR Checklist

@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Bug It either shouldn't be doing this or needs an investigation. Area-Settings Issues related to settings and customizability, for console or terminal Product-Terminal The new Windows Terminal. labels Aug 28, 2024
Copy link
Member

@jazzdelightsme jazzdelightsme left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Member

@lhecker lhecker left a comment

Choose a reason for hiding this comment

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

🎉

@zadjii-msft zadjii-msft merged commit 1482fd4 into main Aug 29, 2024
20 checks passed
@zadjii-msft zadjii-msft deleted the dev/pabhoj/color_selection_ids branch August 29, 2024 20:08
@DHowett
Copy link
Member

DHowett commented Sep 4, 2024

Marked up for 1.22 - thanks for the report and for the quick fix!

@@ -364,7 +403,8 @@
"foreground": "i00",
"background": "i03"
},
"keys": "ctrl+shift+0"
"keys": "ctrl+shift+0",
Copy link
Member

Choose a reason for hiding this comment

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

Hey, does having keys, command and id in the same settings file -- even though it is a SYSTEM settings file -- result in us changing the user's keys list? (e.g. do we accidentally up-convert these key/command/id triples into key/id and action/id pairs?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We do convert these into key/id and action/id pairs in our internal maps, but they're still in the InBox layer so they don't show up in the user's settings file

DHowett pushed a commit that referenced this pull request Sep 4, 2024
## Summary of the Pull Request
Add action IDs to the default commands for color selection

## Validation Steps Performed
Color selection commands now show up in the command palette

## PR Checklist
- [x] Closes #17819
- [ ] Tests added/passed
- [ ] Documentation updated
- If checked, please file a pull request on [our docs
repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx
- [ ] Schema updated (if necessary)

(cherry picked from commit 1482fd4)
Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgSawsY PVTI_lADOAF3p4s4AmhmQzgSwIVc
Service-Version: 1.22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal.
Projects
Development

Successfully merging this pull request may close these issues.

Updated to 1.22.2362.0, and the “Color selection, foreground” commands are MIA
6 participants