Honor open-to-side placement when opening GitHub PR and issue links
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 82/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- github, typescript, vscode
- Domínio
- developer-experience
Direção de pesquisa
Comece em src/github/externalUriOpener.ts e acompanhe como o abridor recebe openContext e chama os métodos de visão geral de PR e issue. Confira em src/github/pullRequestOverview.ts o tratamento de posicionamento existente e, em seguida, adicione testes de regressão para criar e revelar painéis no grupo lateral solicitado. O trabalho estará concluído quando os dois tipos de visão geral respeitarem ViewColumn.Beside e as aberturas comuns mantiverem o comportamento atual.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 2.6k
- Forks
- 805
- Merge médio
- 1d 23h
- PRs com merge (30d)
- 41
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de microsoft/vscode-pull-request-github
-
Issue 3 — DocumentationAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
microsoft/vscode-pull-request-github#8792 ·
Mantenedores costumam responder em até 1 dia
-
Plugin no longer gives access to GitHub issues while using personal account.Talvez já em andamento @alexr00 assumiu há 2 dias. Abertainfo-needed
microsoft/vscode-pull-request-github#9064 · 2 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dependabot PR description is blank in VS Code but renders correctly on GitHubTalvez já em andamento @alexr00 assumiu há 3 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 50/100
microsoft/vscode-pull-request-github#9063 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Configurable issue completion triggerTalvez já em andamento @alexr00 assumiu há 3 dias. Aberta
microsoft/vscode-pull-request-github#9062 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Branch deletion modal (0.168.0): "Delete All" reads as deleting the repository, and single deletes need two tripsTalvez já em andamento @alexr00 assumiu há 3 dias. Aberta
microsoft/vscode-pull-request-github#9060 · 1 comentário · 1 reação · 1 responsável ·
Mantenedores costumam responder em até 1 dia
Todas as issues de microsoft/vscode-pull-request-github
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Chocobozzz/PeerTube#7832 ·
Mantenedores costumam responder em até 3 dias
-
Pay to use?Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 66/100
darkreader/darkreader#16108 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
Add: Dak Nong TV SDAbertacheck:passed streams:add
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
bug engine
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
frostney/GocciaScript#1746 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
agilepathway/label-checker#739 · 4 comentários ·
Mantenedores costumam responder em até 1 dia