Honor open-to-side placement when opening GitHub PR and issue links
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 82/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- github, typescript, vscode
- Lĩnh vực
- developer-experience
Hướng nghiên cứu
Bắt đầu tại src/github/externalUriOpener.ts và lần theo cách opener nhận openContext rồi gọi các phương thức tổng quan PR và issue. Kiểm tra cách xử lý vị trí hiện có trong src/github/pullRequestOverview.ts, sau đó bổ sung kiểm thử hồi quy cho việc tạo và hiển thị panel trong nhóm bên được yêu cầu. Công việc hoàn tất khi cả hai loại tổng quan đều tuân theo ViewColumn.Beside và các lần mở thông thường vẫn giữ nguyên hành vi hiện tại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- TypeScript
- Star
- 2.6k
- Fork
- 805
- Merge trung bình
- 1 ngày 23 giờ
- Pull request đã merge (30 ngày)
- 41
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/vscode-pull-request-github
-
Issue 3 — DocumentationĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
microsoft/vscode-pull-request-github#8792 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Plugin no longer gives access to GitHub issues while using personal account.Có thể đã có người làm @alexr00 đã nhận 3 ngày trước. Đang mởinfo-needed
microsoft/vscode-pull-request-github#9064 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Dependabot PR description is blank in VS Code but renders correctly on GitHubCó thể đã có người làm @alexr00 đã nhận 3 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
microsoft/vscode-pull-request-github#9063 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Configurable issue completion triggerCó thể đã có người làm @alexr00 đã nhận 3 ngày trước. Đang mở
microsoft/vscode-pull-request-github#9062 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Branch deletion modal (0.168.0): "Delete All" reads as deleting the repository, and single deletes need two tripsCó thể đã có người làm @alexr00 đã nhận 4 ngày trước. Đang mở
microsoft/vscode-pull-request-github#9060 · 1 bình luận · 1 reaction · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của microsoft/vscode-pull-request-github
Issue tương tự
-
area:security documentation effort:S priority:P2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
app:web meta:ops
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 78/100
osu-tournament-rating/otr#1113 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
typegoose/mongodb-memory-server#1040 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 92/100