VSCodeVim/Vim

I have overloaded cursorStartPosition to be too many different things.

Aperta

#805 aperta il 27 set 2016

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)TypeScript (1271 fork)batch import
help wantedkind/refactorsize/L

Metriche repository

Star
 (12.664 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Comprendere l'uso corrente di cursorStartPosition e cursorPosition nel codice sorgente, identificare tutti i punti in cui l'intervallo dell'operatore viene mescolato con il tracciamento della selezione, quindi introdurre un nuovo campo su RecordedState per contenere l'intervallo dell'operatore e aggiornare di conseguenza tutta la logica correlata.
Tech stack
typescript
Dominio
tooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50