Cudatext: Poor result using multiple bookmarks

[JairoMartinezA]

Hi @Alexey-T,

1. Start with the next lines:
   

2. Put many bookmarks using the core functionality: bookmarks: put bookmarks on carets, selecting many carets previously.
   

3. Make a selection that includes at least two bookmarks and delete it:
   

4. Cud crash with the remained bookmarks:
   

[Alexey-T]

I don't see a CRASH here. do you mean app crash? or 'incorrect bookmark postion after'? bookmark pos are correct. you deleted block from line 7, so bookmarks from lines 9/14 are moved to line 7. Two marks -> one mark.

[OlehL]

I think will be better delete bookmarks from lines 9/14 when user delete this lines

[JairoMartinezA]

Yeah exactly I prefer to delete all lines with all content all bookmarks.

[JairoMartinezA]

I was validating the usage in Intellij-Idea and yes basically when you delete the lines the associated bookmarks also are deleted.

[Alexey-T]

better delete bookmarks from lines 9/14 when user delete this lines

It was like this before (long ago). it was changed to like it's now. Found!
https://github.com/Alexey-T/CudaText/issues/1241
@kvichans asked to make it like this. Rus text in there.

It's handy. I want to keep bookmarks for big block, even if I delete part of text

[JairoMartinezA]

Hi @Alexey-T

Is possible to give to the user the option of delete all bookmarks or maintain one as now? 😸

[Alexey-T]

will be in next update,

1.117.1 (2020/10)

• add: option "auto_delete_bookmarks"
• fix: on deleting block with n bookmarks, bookmarks shifted upper too much