VSCodeVim/Vim

"go to definition", etc go to the right line, but wrong column

Open

#3.102 aberto em 5 de out. de 2018

Ver no GitHub
 (10 comments) (0 reactions) (0 assignees)TypeScript (1.271 forks)batch import
help wantedkind/bug

Métricas do repositório

Stars
 (12.664 stars)
Métricas de merge de PR
 (Mesclagem média 60d 20h) (17 fundiu PRs em 30d)

Description

Describe the bug using "go to definition" doesn't seem to update the column - it keeps the previous column.

To Reproduce Steps to reproduce the behavior:

  1. Click on a function call
  2. Right-click
  3. "go to definition"
  4. Notice the column LOOKS correct
  5. Move cursor using hjkl
  6. Notice the cursor column jumps to where you were before the "go-to-definition", but on the correct line.

Expected behavior The column is in-sync just like the row

Screenshots

Environment (please complete the following information):

  • Extension (VsCodeVim) version: 0.16.6
  • VSCode version: 1.27.2
  • OS: Linux - Kubuntu current LTS

Additional context None

Guia do colaborador

Direção de pesquisa
Investigue como o VSCodeVim atualiza a coluna do cursor após executar 'ir para definição'. Veja o manipulador do comando 'ir para definição' e rastreie como a posição do cursor é definida. O problema sugere que apenas a linha é atualizada, não a coluna. Verifique as chamadas da API do VSCode como `vscode.commands.executeCommand('editor.action.goToDefinition')` e a lógica subsequente de posicionamento do cursor na extensão.
Pilha de tecnologia
typescript
Domain
developer experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65