VSCodeVim/Vim

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

Open

#3 102 ouverte le 5 oct. 2018

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)TypeScript (1 271 forks)batch import
help wantedkind/bug

Métriques du dépôt

Stars
 (12 664 stars)
Métriques de merge PR
 (Merge moyen 9j 13h) (7 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Enquêtez sur la façon dont VSCodeVim met à jour la colonne du curseur après avoir exécuté 'Aller à la définition'. Regardez le gestionnaire de la commande 'Aller à la définition' et tracez la façon dont la position du curseur est définie. Le problème suggère que seule la ligne est mise à jour, pas la colonne. Vérifiez les appels API VSCode comme `vscode.commands.executeCommand('editor.action.goToDefinition')` et la logique de positionnement du curseur qui suit dans l'extension.
Stack technique
typescript
Domaine
developer experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65