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

Word-level diff behavior #85

Closed
lujuste opened this issue Oct 20, 2022 · 3 comments · Fixed by #89
Closed

Word-level diff behavior #85

lujuste opened this issue Oct 20, 2022 · 3 comments · Fixed by #89

Comments

@lujuste
Copy link

lujuste commented Oct 20, 2022

diff doesn't make a difference at the word level but at the character level, and this ends up generating some semantic problems and causing bugs, would there be any way to change the word itself and not the letter?

For example using this logic:

https://codepen.io/percipient24/pen/eEBOjG

It is possible to notice that when there is an equal character, this character is ignored as a change. Many of the words get confused whenever there is an equal character in the middle of the word that I want to remove entirely.

I would like to know if it is possible to just change the word and mark the word and not the character level. Which way could I go?

In the example below I don't want it like this:

wrong png

but I would like it to be like this:

right

Can someone please help me find a way to replace words and not letters? I love quill js and I really need to implement this functionality

@lujuste
Copy link
Author

lujuste commented Oct 20, 2022

Please guys anyone can be save me?

@luin
Copy link
Member

luin commented May 19, 2023

Hey @lujuste 👋 ,
Sorry for the delay! Now word level diffing is the default behavior.

@lujuste
Copy link
Author

lujuste commented May 19, 2023

@luin tks for reply! great job!
I will test! Thank you very much

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 a pull request may close this issue.

2 participants