Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug when working backwards in multiple-cursors #6

Open
obar opened this issue Feb 24, 2020 · 1 comment
Open

Bug when working backwards in multiple-cursors #6

obar opened this issue Feb 24, 2020 · 1 comment

Comments

@obar
Copy link

obar commented Feb 24, 2020

This is a brilliant interface! Thanks for the great package.

It even works with multiple-cursors… except when extending backward. Not a deal-breaker as one can move the cursors to do it if need be, but thought I should record this bug.

Here's a demo of what happens with three lines, and three cursors at the start of 'over'.

The quick brown fox jumps ^over the lazy dog.
The quick brown fox jumps ^over the lazy dog.
The quick brown fox jumps ^over the lazy dog.

Then type this sequence:
M-u M-u M-u C-b

The quick brown fox jumps OVER THE LAZY^ dog.
The quick brown fox jumps OVER THE LAZY^ dog.
The quick brown fox jumps OVER THE LAZY^ dog.

M-l

The quick brown fox jumps OVER THE lazy^ dog.
The quick brown fox jumps OVER THE lazy^ dog.
The quick brown fox jumps OVER THE lazy^ dog.

M-l

The quick brown fox jumps OVER the lazy^ dog.
The quick brown fox jumps over THE lazy^ dog.
The quick brown fox jumps OVER THE lazy^ dog.

It looks to be caused by the use of fix-word--quick-fix-times as a global variable. Perhaps there would be a way to increment that count only once, or divide by the number of active cursors, or look into how smartparens handles sp--mc/cursor-specific-vars… haven't dug into this too much, as I said any case switching can be performed in other ways right now.

@mrkkrp
Copy link
Owner

mrkkrp commented Feb 24, 2020

I'd accept a PR fixing this, but I myself do not use multiple cursors and do not have time to experiment with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants