Fix copy link context menu for issue overview#8414
Conversation
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
alexr00
left a comment
There was a problem hiding this comment.
We don't want to show the vscoddev option for the issues view.
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Removed the vscode.dev option from the issue overview context menu in 9c5a98e. Now only "Copy Link" appears for issues, while PRs still show both options. |
There was a problem hiding this comment.
Pull request overview
This PR fixes a bug where the #123 link in the issue overview title wasn't copyable via the right-click context menu. The fix extends the existing pr.copyPrLink command to also work with issues by adding support for the IssueOverview webview and implementing a fallback to resolve issues when pull request resolution fails.
Changes:
- Extended the
pr.copyPrLinkcontext menu command to work in both PullRequestOverview and IssueOverview webviews - Updated the command handler to fall back to
resolveIssuewhenresolvePullRequestreturns undefined - The "Copy vscode.dev Link" option remains PR-only as intended
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Added IssueOverview to the webview ID condition for the pr.copyPrLink context menu item |
| src/commands.ts | Refactored command handler to support both PRs and issues by adding fallback logic and updating variable types |
621e7fb to
9c5a98e
Compare
|
Was hoping that force pushing back to the commit that had been reviewed would prevent this from needing re-review. Alas. |
#123link in the issue overview title is not copyable because context menu items for "Copy Link" are only registered forPullRequestOverviewwebview, notIssueOverviewpackage.jsonto add theIssueOverviewwebview ID to the context menu registration forpr.copyPrLinkonly (notpr.copyVscodeDevPrLink)commands.tsto resolve issues (not just PRs) when copying links forpr.copyPrLinkpr.copyVscodeDevPrLinkas PR-only per review feedbackitemtoprOrIssuefor better readabilityOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.