Pasting issues

[bjhess]

With the recent improvements and bugfixes on pasting, it seems a new bug has appeared. On 0.16.3 I noticed weird behavior pasting with the TipTap YouTube extension (a paste leads to both the link and the embed appearing) and some odd behavior with our custom Markdown processing paste extension.

So I wondered if regular text pasted as expect? It appears not. You'll notice that the text in my example here gets pasted twice, and the duplicate version of the paste seems to have newlines stripped or something of that nature.

CleanShot.2025-02-06.at.19.25.33.mp4

I'm guessing if the text-pasting bug is fixed it will also resolve the bugs with pasting extensions.

Thank you!

[KonnorRogers]

🤔 perhaps these 2 need to return true, ill take a look.

[KonnorRogers]

Doh. never linked the lines, but i think i found the issue regardless.

Fix here:

#261

[bjhess]

Unfortunately while now the double-paste of regular text is fixed, my paste-based extensions are now not functioning again. Is there anything I can provide to help troubleshoot the paste-based extension interruption?

[KonnorRogers]

@bjhess ideally one of the failing extensions would help tremendously

[bjhess]

The failing extensions are:

• YouTube
• This pasted Markdown extension we have in our repo to process pasted Markdown

If you need me to try to bring one or more of those into a Rhino branch, I can, but I might not be able to get to it until next week. Thank you!

[bjhess]

I did try to add the YouTube extension to my fork of rhino-editor tonight, but I couldn't quite crack it. No errors, but the extension didn't seem to be doing anything, so I probably did it wrong.

[KonnorRogers]

@bjhess I think I know the problem now. I think I know whats happening.

https://prosemirror.net/docs/ref/#view.EditorProps.handlePaste

Can be used to override the behavior of pasting. slice is the pasted content parsed by the editor, but you can directly access the event to get at the raw content.

It seems like dispatching a custom event like im currently doing isn't the greatest way since the function is synchronous so no way to check defaultPrevented on events that are checked higher in the bubbling. So I think handlePaste is really bad for being compatible with other editors.

I think I need to move all the transforming code into a transformPastedHTML hook, and make sure it always runs last to allow plugins to override. In addition, rhino-paste can not be relied upon to actually handle paste operations like it does today in rhino, but rather, is just a way to know if a user pasted something. I think I have a clear path forward to fix this.