Syméon

Results 54 comments of Syméon

You are correct about the issue. The issue is that we are missing information, I can fix the undo issue for the replace by doing: ```Java private static void moveToChange(...

I'm working on UT the `PlainTextChange` before adding data in `TextChange` to make sure nothing will break. Especially that `merge` method which is rather complex and which I'm trying to...

@Jugen I'm adding some UT to some parts of the code and I found something that might be incorrect. Now, my experience when working with legacy code like this one...

It appears that placing the caret within `TextChange` or its subclass `RichTextChange` isn't an option, since these are part of the document model and inherently unaware of the caret's position....

@Jugen I pushed a PR. If we could get that change in a build that would be most appreciated. Note that after all these changes, the one that I really...

@Jugen would it be possible to get a build with that simple fix? I would like to integrate in my release. Also, is there a way to get the right...

@Jugen thanks. I hope I didn't overwhelm you with the details in this ticket. I noticed you closed it, but there are still a few issues that are pending with...

Ok, but you get the general idea of my comments: 1. Split methods to keep them short 2. Have significant naming for variables and methods Breaking in small methods is...

I'm having a look but would you have the context from which your code is being called. It is very likely the issue you are facing is in the way...

I don't think there is an issue with the library, it is more a question for StackOverflow or similar forums as you are having issues debugging your code. That being...