BUG: Duplicated links are added using rte

[yashvi2026]

GrapesJS version

• I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome

Reproducible demo link

https://codepen.io/Harshsne/pen/xxoOyXO

Describe the bug

How to reproduce the bug?

1. add link to a word using rte, link component is added
2. Now type something after the link component without space and select that word again add link to it .
3. after adding it ,click outside the textbox.

What is the expected behavior?
Second Added link should be wrapped around the first link and the text.

What is the current behavior?
Two link component is added.

code
var url = window.prompt('Enter the URL to link to:');
rte.insertHTML(<a class="link" href=${url}>${rte.selection()}</a>);

Code of Conduct

• I agree to follow this project's Code of Conduct

[sirbeagle]

So I've been playing around with this quite a bit and I can reproduce the error on the Codepen that @yashvi2026 provided. My steps to reproduce (most of the time) are very similar:

1. Type two words with numerous spaces in between in a text box.
2. Highlight the first word by double-clicking word or manually choose with mouse
3. Use the custom RTE to add a link
4. Click outside the box
5. Double click back after the link in the existing text
6. Highlight the word after the link and extend that highlight to include the first letter of the first link using a mouse
7. Use RTE to add another link
8. Click outside of the text box

However, I was unable to reproduce this in a dev environment. After lots of trial and error, I saw that the Codepen is using GrapesJS v0.22.1. I switched that to use v0.22.4 (the latest Codepen could find) and I could no longer reproduce it.

@yashvi2026 - if you upgrade the version in your Codepen (or wherever else you were seeing this) can you still reproduce the error?

[yashvi2026]

Yes.After upgrading to latest version ,this issue doesnt occur.

[sirbeagle]

Alight - the only bad part is we may never know what the fix ended up being.

@artf - I think we can close out this issue.