If you select text, and paste a URL from the clipboard, auto-link the selected text rather than replace it with the URL

[matt]

The summary kind of says it, but I think this is a text editor interaction pattern I'm seeing more and more now that is pretty addictive once you use it, and makes a very common task much faster.

I think this would be a groovy enhancement to our WYSIWYG, but isn't urgent or a blocker for anything.

[iseulde]

I like this idea. :)
Works for images too.

[azaozz]

31571.patch is looking good. Only thinking we should look at what exactly is selected and check that the selection is not inside a link already.

It is possible to select several paragraphs and turn them into a link, not sure we want that whit this method. Also, we (probably) want to support that when an image is selected, need to exclude placeholder images, etc.

[iseulde]

Also, we (probably) want to support that when an image is selected, need to exclude placeholder images, etc.

That's right, though we should probably prevent any commands for these, just like wpviews.

[iseulde]

Only thinking we should look at what exactly is selected and check that the selection is not inside a link already.

So if there's a link already, it should insert, not update? I think updating is fine. :)

[iseulde]

This will prevent the execution of any commands on a placeholder.

[azaozz]

In 31691:

TinyMCE: when pasting an URL over a selection, insert a link with the URL instead of replacing the selection with it. Props iseulde. Fixes #31571.

[iseulde]

We can't ignore all commands. Toggling the toolbar is impossible now if a placeholder is selected. We'll probably need to handle this like the views.

[iseulde]

Played a bit with the formatter and data-mce-contenteditable="false", but there doesn't seem to be an easy way to ignore certain elements. So better to remove that part. After all placeholders could be linked before using the button.

[iseulde]

And again #29804 would make this better. :)

[azaozz]

In 31738:

TinyMCE: remove blocking of commands on placeholder images. Breaks some execCommand.
Props iseulde. Fixes #31571.