Fix inline paste

[ellatrix]

Description

Fixes #3305. Broken since fecc4b3.

How Has This Been Tested?

Paste some pure text in a paragraph. Text should be inline. It's not, a new block is created.
Or, paste a link on a work. Word should be linked. Instead a new block with the link is created.

[ellatrix]

Maybe we can make this a bit more readable... :)

[mcsf]

This, along with a private chat, shed some context for me on rawHandler's inline option. Note that the @param is wrongly typed, as it reads Boolean:

https://github.com/WordPress/gutenberg/blob/b47cbf52c05dd9450e24f8cafabfebcd04023f10/blocks/api/raw-handling/index.js#L28-L36

[mcsf]

Maybe we can make this a bit more readable... :)

Indeed, if inline can encode more than two states, we should improve this. Ideas:

• Plain enumeration: param takes values in HANDLER_AUTO, HANDLER_BLOCK, HANDLER_INLINE. IMO the most direct and plausible way to address this.
• Masks: same idea, but if we need variations of state, we can combine with bitwise operations. Not idiomatic of JS, and likely confusing.
• Class interface: param needs to be instanceof HandlerMode, created with new AutoHandlerMode(), new BlockHandlerMode(), etc. Offers some type safety, but seems very cumbersome, over-the-top, and also bulky for an interpreted language.
• Algebraic data types: oh wait, this is JS.

[mcsf]

Let's revisit rawHandler soon. For now, we can get this fixed.

[ellatrix]

For the first point, would the parameter name stay inline?

[mcsf]

For the first point, would the parameter name stay inline?

In my mind it would be more something like handleMode.