VSCodeVim/Vim

I have overloaded cursorStartPosition to be too many different things.

Open

#805 aberto em 27 de set. de 2016

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)TypeScript (1.271 forks)batch import
help wantedkind/refactorsize/L

Métricas do repositório

Stars
 (12.664 stars)
Métricas de merge de PR
 (Mesclagem média 9d 13h) (7 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Entenda o uso atual de cursorStartPosition e cursorPosition no código, identifique todos os lugares onde o intervalo do operador é misturado com o rastreamento de seleção, então introduza um novo campo no RecordedState para conter o intervalo do operador e atualize toda a lógica relacionada de acordo.
Pilha de tecnologia
typescript
Domain
tooling
Tipo Issue
Refatoração
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
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50