Honor open-to-side placement when opening GitHub PR and issue links
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 82/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- github, typescript, vscode
- Bereich
- developer-experience
Rechercherichtung
Beginne in src/github/externalUriOpener.ts und verfolge, wie der Opener openContext erhält und die Methoden für PR- und Issue-Übersichten aufruft. Prüfe in src/github/pullRequestOverview.ts die vorhandene Handhabung der Platzierung und füge anschließend Regressionstests für das Erstellen und Anzeigen von Panels in der angeforderten Seitengruppe hinzu. Das Ziel ist erreicht, wenn beide Übersichtstypen ViewColumn.Beside berücksichtigen und gewöhnliche Öffnungen ihr bisheriges Verhalten beibehalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Summary
Opening a GitHub PR link with an explicit open-to-side request still opens/reveals the PR overview in the current editor group. The GitHub Pull Requests external URI opener receives a placement request from VS Code but does not forward it to the overview panel.
This was reproduced in a live Code OSS development instance using the Quick Open link support in https://github.com/microsoft/vscode/pull/342081 (implements https://github.com/microsoft/vscode/issues/342051).
Steps to reproduce
- Use a VS Code build with Quick Open link support and external-opener placement context, and enable the GitHub Pull Requests link handler.
- Sign in/authorize GitHub Pull Requests. Keep an existing document open in a single editor group.
- Press Ctrl+P and enter
https://github.com/microsoft/vscode/pull/342081. - Accept the Open Link result with Alt+Enter (an explicit open-to-side gesture).
Expected: The GitHub Pull Requests overview opens beside the originating document, preserving that document in its group.
Actual: The PR overview opens in the same editor group. No side group is created. The user also observed the same placement problem with Alt+click; the instrumented reproduction below specifically used Alt+Enter in Quick Open.
Live dbgjs evidence
A non-pausing breakpoint at MainThreadUriOpeners.createOpener recorded the context immediately before the call into the extension host:
{
"id": "GitHub.vscode-pull-request-github.issueOrPullRequest",
"uri": "https://github.com/microsoft/vscode/pull/342081",
"viewColumn": -2
}
-2 is ViewColumn.Beside. After the request, the workbench still had one editor group, containing the original Untitled-1 tab and the #342081 Support opening... PR tab.
This demonstrates that Quick Open and the workbench external-opener pipeline are forwarding the side-placement request; the PR extension is not honoring it. The temporary breakpoint was removed after tracing.
Code pointers / suggested fix
src/github/externalUriOpener.ts:37receives the open context, but currently usessourceUriwithout consuming the placement field supplied by the host.- The PR overview call at line 63 and issue overview call at line 72 omit placement.
PullRequestOverviewPanel.createOrShowat line 123 already has placement handling viatoTheSide/_getViewColumn.
Please honor openContext.viewColumn === vscode.ViewColumn.Beside when supplied by a supporting VS Code host, and forward the request for both PR and issue overviews. Preserve ordinary opening behavior when no side placement is requested. Include regression coverage for both creating a panel and revealing an existing panel in the requested side group. Issue-overview impact is inferred from the shared source path; the live reproduction was for a PR.
Environment
- Windows, Code OSS development build, VS Code branch used by microsoft/vscode#342081.
- Live placement trace: GitHub Pull Requests 0.170.0 (stable).
- The installed prerelease 0.171.2026101604 was also inspected; its URI opener likewise did not forward placement. Separate authentication/rendering problems encountered during setup are not the subject of this report.
Related navigation story: https://github.com/microsoft/vscode/issues/342050
Related Alt+click request: https://github.com/microsoft/vscode/issues/340724
- Vorherrschende Sprache
- TypeScript
- Sterne
- 2.6k
- Forks
- 805
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 41
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus microsoft/vscode-pull-request-github
-
Issue 3 — DocumentationOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
microsoft/vscode-pull-request-github#8792 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Plugin no longer gives access to GitHub issues while using personal account.Evtl. vergeben @alexr00 hat das vor 3 Tagen übernommen. Offeninfo-needed
microsoft/vscode-pull-request-github#9064 · 2 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Dependabot PR description is blank in VS Code but renders correctly on GitHubEvtl. vergeben @alexr00 hat das vor 3 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 50/100
microsoft/vscode-pull-request-github#9063 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Configurable issue completion triggerEvtl. vergeben @alexr00 hat das vor 3 Tagen übernommen. Offen
microsoft/vscode-pull-request-github#9062 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Branch deletion modal (0.168.0): "Delete All" reads as deleting the repository, and single deletes need two tripsEvtl. vergeben @alexr00 hat das vor 4 Tagen übernommen. Offen
microsoft/vscode-pull-request-github#9060 · 1 Kommentar · 1 Reaktion · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in microsoft/vscode-pull-request-github
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
jbacalso24/mimry#459 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
architecture help wanted hive/claimed-by-architect origin/architect stage/030-pr-in-progress tech-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
kubestellar/console#24422 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area: backends type: docs
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 83/100
schlessera/brain-kit#2383 ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent duplicate providers tool triaged
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag