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

Multiple replacement on the same line is deleting texte before cursor #262

Closed
jdutriaux opened this issue Mar 17, 2015 · 9 comments
Closed

Comments

@jdutriaux
Copy link

Hi, using contenteditable I have a bad behavior, see the image below :

Gif

@ichord
Copy link
Owner

ichord commented Apr 13, 2015

Sorry not to reply in time, for it's a difficulty bug and i don't have much time to fix it;
And i'm sorry to tell that i might not fix it if not much people get this bug, for it's hard to fix and not "usual"

@jdutriaux
Copy link
Author

Oh okay :(.

Thanks for the answer and for the awesome work but that bug makes the library unusable with contenteditable, a pity because I really like your library.

@jdutriaux
Copy link
Author

Sorry but I don't understand your comment @houd1ni :(

This bug can happen when you are many lines under a replaced content.

@houd1ni
Copy link

houd1ni commented May 12, 2015

@MrHalfman I saw and then removed comment at once. Will think about that issue nearest time 😉

@jdutriaux
Copy link
Author

Thanks! 😄

@houd1ni
Copy link

houd1ni commented May 12, 2015

@MrHalfman done it. Wait for pull request and accept 😄

@jdutriaux
Copy link
Author

Many thanks to you 👏 @houd1ni !

@houd1ni
Copy link

houd1ni commented May 12, 2015

@MrHalfman 😉 come here often. Hope, Harold will too. I make own communication app and just don't want to propogate branches. Maybe, then I'll add some functions of earlier work for that app to merge it, if Harold agree, of cource 😈

ichord added a commit that referenced this issue May 17, 2015
one approach to fix #258 and elegant #262
@ichord ichord closed this as completed May 17, 2015
@ichord
Copy link
Owner

ichord commented May 17, 2015

reopen for not solved perfectly.

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

No branches or pull requests

3 participants