Fix follow for multiple cursors#1111
Conversation
|
Why would you ever want to follow multiple cursors? |
|
I do not know. In my app I just want to follow one of them but we did allow to configure several each with its follow config and I guess we should be consistent. Another possibility is to remove the independent configuration and state that we just follow the first one. |
|
The current implementation follows all of them. Setting the focus to the next one that moves. |
|
There are posible solutions but the current one makes no sense. All cursors are followed when the global option is set. |
|
Yeah, if it's possible to follow any cursor other than the main cursor, it makes sense to let the developer choose which cursor to follow. So the .follow boolean looks good to me, simple and hopefully working without issues. |
You are right, and makes more sense. But in order to make does work I had to change the default to true when options are not provided (case of one cursor). Done and ready to merge!!!! |
@sschmidTU testing with the migration I have realised that the independent control of follow for multiple cursors was not working.
Do you agree with the approach?