Drag and Drop Media prevents Drag and Drop Text

[Ipstenu]

Related to #27465

Drag-and-drop text (that is selecting text and dragging it to the text editor window) no longer works in WordPress due to the new Drag & Drop Media feature. This is the case in the HTML and Visual editors. This has been reported in the WordPress.org forums by a few people.

​https://wordpress.org/support/topic/text-editor-ruined-in-39

[azaozz]

Not sure how common is this. Testing on Win7, it will only let me drag selected text from one web browser to another, not from a text editor. It's quite inconvenient to drag too.

To fix, seems we will have to bring back something like isDraggingFile() from https://core.trac.wordpress.org/attachment/ticket/19845/19845.8.diff and throttle it down as there are some performance issues. Or maybe set a draggingFile var the first time we see a drag event and reset it when the dragging has finished.

[Ipstenu]

It's hard to gauge. I think it's one of those 'Heavily used by people who knew about it' features. I find it useful in forums or tickets when I want to quote, but I'm really a little surprised it's in-use for writing posts.

Would it be possible to have an option (define option, not a check box) to allow people to turn off the whole-page drag/drop to get it back? That MIGHT get us a better idea of how it's used.

[kraftbj]

Agreeing with Ipstenu, would it make sense to add a filter for the default args for the Edit Posts editor and suggest a plugin for the previous behavior to save the performance hit of isDraggingFile()?

Attaching patch and example plugin in a moment.

[kraftbj]

Add a filter for the default args

[kraftbj]

Example plugin for old behavior based on 27880.diff

[azaozz]

Not sure if another filter is needed there. If you want to disable drag/drop uploading from a plugin, this should do it:

add_action( 'after_wp_tiny_mce', 'my_disable_ddupload' );
function my_disable_ddupload() {
	?>
	<script>
	if ( typeof tinyMCEPreInit !== 'undefined' ) {
		delete tinyMCEPreInit.dragDropUpload;
	}
	</script>
	<?php
}

[Zumatra]

I'm not even sure I am allowed to post here as I am in no way involved in WP development but I was following a link from a thread I started on this subject.

I think that to be a little surprised that drag and drop is used in writing posts kind of suggests a narrow scope of how WP is used and that it is only used for blogging. As somebody else said, I used it to quote chunks of text and YouTube descriptions of videos which I embed. It saves a lot of time when one spends a couple of hours per day posting and the extra time spent soon mounts.

I definitely cannot drag and drop from one browser to another - at least not from Safari to Firefox because that's how I first discovered the problem. I can sometimes (but I haven't discovered the necessary conditions to do so) drag and drop from Mac TextEdit to Safari but only if I place the cursor at the text insertion point. It may be that text must have already been inserted but Drag and Drop from TextEdit to Safari isn't something I do very often - it's usually Safari-Firefox or Safari-Safari, neither of which work.

Replying to Ipstenu:

It's hard to gauge. I think it's one of those 'Heavily used by people who knew about it' features. I find it useful in forums or tickets when I want to quote, but I'm really a little surprised it's in-use for writing posts.

Would it be possible to have an option (define option, not a check box) to allow people to turn off the whole-page drag/drop to get it back? That MIGHT get us a better idea of how it's used.

[azaozz]

27880.2.diff fixes it by looking at what is being dragged. It adds a bit of overhead especially while dragging in TinyMCE as it passes the normalized jQuery event to the parent document, but that is roughly the equivalent of attaching the events with jQuery in the iframe.

[azaozz]

In 28189:

Drag/drop on the editor to upload: don't trigger the uploader when selected test is being dragged from one window to another. See #27880, for trunk.

[nacin]

In 28262:

Drag/drop on the editor to upload: don't trigger the uploader when selected text is being dragged from one window to another.

Merges [28189] to the 3.9 branch.

props azaozz.
fixes #27880.