clipboardTextParser(text, $context)?

1

Hi,

i’m trying to handle this scenario:

  • a user copies an embed code (typically, on youtube)
  • then pastes it into the editor

However, the pasted snippet is html markup pasted as text/plain.

clipboardTextParser(text) -> slice would do the job, but it cannot parse the text properly because it’s missing the $context variable.

Would you accept a PR with this change ?

2

Here’s the PR: https://github.com/ProseMirror/prosemirror-view/pull/22

3

Yes, that makes sense. Merged!

4

Cool !

However, having no less than FIVE hooks in parseFromClipboard looks like suspicious design.

Couldn’t it be simplified ?

5

Also to make the conversation easier, here’s what i did that motivated the pull request:

6

Do you have any specific proposal?

7

I don’t know, maybe simply allow overriding the whole parseFromClipboard function ?

8

That would make it impossible to ‘mix in’ transformers from different addons, and would require people overriding it to get all the details, such as pasting into code and creating the proper slice, right in their implementations. If you want to completely redefine paste behavior, you can already do that with handlePaste.