Wedge
Public area => Bug reports => The Pub => Archived fixes => Topic started by: billy2 on July 5th, 2012, 09:16 AM
-
Select a line of text and try to move it to different point within the post. This action brings up the 'add attachment' box rather than drop the text at the new insertion point.
Cheers
-
Uh, I've actually never done this in my life... Didn't even think it was possible :P
I'd usually use cut & paste for that, and I'd recommend that you'd do it as it's faster, ah ah...
Anyway... Is it worth 'fixing'? Is it even possible to check that a drag is coming from a textarea...? (Heck, from a specific class even...)
-
Worth fixing? Not for me Nao.
But for those that type reams of replies ??
:eheh:
-
I do that, but I don't move text blocks around...
-
Works for me in IE8.
-
Sorry?
What is IE ?
:D
-
That's very likely related to the attachments plugin and probably wouldn't occur in base Wedge.
-
Yup, but OTOH I have plans to make it core... :^^;:
-
In which case we'll need to figure out why it's grabbing events it shouldn't be ;)
-
Drag & drop is a strange beast really...
I noticed that it DOES have a meaning overall, though, it's just that it needs some adapting.
-
Still doing it, btw.
-
Events aren't funny...
I guess the plugin will stay a plugin for now... <sigh>
-
I think I've fixed it... please review this small patch https://github.com/live627/MultiAttach/commit/59bea5ff6ccc53836e922e92a318b887fae4f4cb
-
Wouldn't this check be better? (Same thing is used in the drop event)
// Make sure we are dragging a file over
if (!e.originalEvent.dataTransfer && !(dt.files || (!$.browser.webkit && e.originalEvent.dataTransfer.types.contains && e.originalEvent.dataTransfer.types.contains('Files'))))
return false;
EDIT: I just realised this thing is months old and I have no clue why this was opened :P