BUG: Text disappears if pasted for a second time in text block

varadero commented 2 years ago

GrapesJS version

[X] I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome 99

Reproducible demo link

https://grapesjs.com/demo.html

Describe the bug

How to reproduce the bug?

Open https://grapesjs.com/demo.html and delete all sample sections
Place one "Text section"
Open a text editor and type 3 lines of text. Something like this:
```
Line 1
Line 2
Line 3
```
Copy the 3 lines from the text editor
Go back to GrapesJS demo page and double click the text section just before the word consecterur (other word would work too)
Press CTRL+V to paste the text
Click outside to commit the changes
Hover your mouse over the Line 2 and notice it is its own Text element
Double click on Line 2 just before the 2
Press CTRL+V to paste the same 2 lines text
Click outside to commit the changes

What is the expected behavior? The text that was pasted the second time should be kept where it was pasted

What is the current behavior? The text that was pasted the second time disappears

If is necessary to execute some code in order to reproduce the bug, paste it here below:

// your code here

Screenshots of the steps:

Go to https://grapesjs.com/demo.html and delete all sample sections. Put one "Text section":
Open text editor, type 3 lines and copy them. Double click on the text in the text section and move the caret just before the word consecterur and press CTRL+V to paste the 3 lines and click outside to commit the changes:
Double click on the second line and press CTRL+V again:
Now click outside to commit the second paste - the last pasted lines disappear:

Code of Conduct

[X] I agree to follow this project's Code of Conduct

varadero commented 2 years ago

I can confirm that this problem occurs only with Grapesjs 0.18.3 - I downgraded to 0.18.2 and the problem does not occur. Older versions are OK too.

jmaks1 commented 2 years ago

@artf The problem is relevant to the latest version. (v0.19.4)

The bug occurs if you copy and paste a nested component while editing text. A regular paste (cmd + v) copies all html, including id, and after removing focus, html with identical id is deleted.

jmaks1 commented 2 years ago

I can confirm that this problem occurs only with Grapesjs 0.18.3 - I downgraded to 0.18.2 and the problem does not occur. Older versions are OK too.

in version 0.18.2, nested components were not created inside the text.

MaxaTech commented 2 years ago

@artf +1

@artf The problem is relevant to the latest version. (v0.19.4)

The bug occurs if you copy and paste a nested component while editing text. A regular paste (cmd + v) copies all html, including id, and after removing focus, html with identical id is deleted.

GrapesJS / grapesjs