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

Make MouseMultiCursor toggle cursors #3146

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

dmaluka
Copy link
Collaborator

@dmaluka dmaluka commented Feb 20, 2024

It is useful to be able to use mouse not only for adding new cursors but also for removing them. So let's modify MouseMultiCursor behavior: if a cursor already exists at the mouse click location, remove it.

It is useful to be able to use mouse not only for adding new cursors
but also for removing them. So let's modify MouseMultiCursor behavior:
if a cursor already exists at the mouse click location, remove it.
@zyedidia zyedidia merged commit e5026ef into zyedidia:master Mar 4, 2024
3 checks passed
shkschneider pushed a commit to shkschneider/macro that referenced this pull request Mar 5, 2024
It is useful to be able to use mouse not only for adding new cursors
but also for removing them. So let's modify MouseMultiCursor behavior:
if a cursor already exists at the mouse click location, remove it.
@Gavin-Holt
Copy link

Hi

Sorry this does not seem to work on my Windows system.

OS: Windows 10
Version: 2.0.13
Commit hash: 68d88b5
Compiled on April 20, 2024

Kind Regards Gavin Holt

@dmaluka
Copy link
Collaborator Author

dmaluka commented May 1, 2024

Which is not surprising: 2.0.13 was released 4 months before this was implemented.

You'd need to compile newest micro from source.

@dmaluka
Copy link
Collaborator Author

dmaluka commented May 1, 2024

Compiled on April 20, 2024

Hmm, 2.0.13 but compiled on April 20?
Are you maybe using a nightly build? There is an issue with nightly builds, which I mentioned today in #3276 (comment)

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

Successfully merging this pull request may close these issues.

3 participants