VSCodeVim/Vim

I have overloaded cursorStartPosition to be too many different things.

Open

#805 ouverte le 27 sept. 2016

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

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

Currently cursorStartPosition and cursorPosition can mean both "the start and end of the current selection" and "the range over which to apply an operator".

These are actually totally different! It's especially bad when you apply an operator and then suddenly you lose the tracked position of the cursor.

Move "the range over which to apply an operator" into a new field on RecordedState and use that instead of cursorStartPosition and cursorPosition. Continue using cursorStartPosition and cursorPosition for selections and cursor locations.

My current problem is running ci( on |(abcd), which promptly forgets the position of the cursor and can't (because of the overloading) figure out where to place it.

Guide contributeur

Direction de recherche
Comprenez l'utilisation actuelle de cursorStartPosition et cursorPosition dans le code, identifiez tous les endroits où la plage de l'opérateur est mélangée avec le suivi de sélection, puis introduisez un nouveau champ sur RecordedState pour contenir la plage de l'opérateur et mettez à jour toute la logique associée en conséquence.
Stack technique
typescript
Domaine
tooling
Type d'issue
Refactorisation
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50