VSCodeVim/Vim

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

Ouverte

#3 102 ouverte le 5 oct. 2018

Voir sur GitHubGuide contributeur
 (10 commentaires) (0 réaction) (0 personne assignée)TypeScript (1 271 forks)batch import
help wantedkind/bug

Métriques du dépôt

Stars
 (12 664 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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