Skip to content

Conversation

wojd0
Copy link

@wojd0 wojd0 commented Jul 8, 2025

closes #2235

@wojd0
Copy link
Author

wojd0 commented Jul 8, 2025

demo:

copy.vscode.demo-1.mp4

I'll continue to work on this for IntelliJ side

@MaxKless
Copy link
Collaborator

Can't watch the video yet but good job! I'm thinking, maybe it's better to have this as a copy button on the quickpickitem instead of as a separate item? It seems like a cool api that we could use here to add the functionality without making things noisier. What do you think?
image

Copy link
Contributor

nx-cloud bot commented Jul 14, 2025

View your CI Pipeline Execution ↗ for commit d0be253

Command Status Duration Result
nx affected --targets=lint,test,build,e2e-ci,ty... ✅ Succeeded 4m 50s View ↗
nx-cloud record -- yarn nx sync:check ✅ Succeeded 3s View ↗
nx-cloud record -- ./gradlew :intellij:ktfmtChe... ✅ Succeeded 3s View ↗
nx-cloud record -- ./gradlew projectReportAll ✅ Succeeded 7s View ↗
nx affected --targets=build,test,e2e-ci --confi... ✅ Succeeded 7m 58s View ↗

☁️ Nx Cloud last updated this comment at 2025-07-14 12:15:52 UTC

@wojd0
Copy link
Author

wojd0 commented Jul 14, 2025

@MaxKless this would be a great idea, definitely much better solution for this feature. However there is one problem - as shown in the docs screenshot you posted, QuickInputButtons requires the QuickPick to be created using createQuickPick API instead of showQuickPick which we currently use.

const selection = await window.showQuickPick(flagItems, {

If we want to go for it, it would require a breaking change to the createQuickPick API.
https://code.visualstudio.com/api/references/vscode-api#QuickPick<T>

@MaxKless
Copy link
Collaborator

I think we can just change the function to use createQuickPick instead of showQuickPick, it shouldn't be a problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to copy constructed command to command palette
2 participants