Skip to content

Commit

Permalink
remove jQuery.event.props usage
Browse files Browse the repository at this point in the history
 use e.originalEvent.dataTransfer as recommended (jquery/jquery#3112) event.props throws error on jQuery 3
 tested on jQ1.11.2, 2 and 3
  • Loading branch information
Matthijs Alles committed Jun 16, 2016
1 parent b734da4 commit 02d5f2c
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/js/components/upload.js
Expand Up @@ -43,15 +43,15 @@

this.on("drop", function(e){

if (e.dataTransfer && e.dataTransfer.files) {
if (e.originalEvent.dataTransfer && e.originalEvent.dataTransfer.files) {

e.stopPropagation();
e.preventDefault();

$this.element.removeClass($this.options.dragoverClass);
$this.element.trigger('dropped.uk.upload', [e.dataTransfer.files]);
$this.element.trigger('dropped.uk.upload', [e.originalEvent.dataTransfer.files]);

xhrupload(e.dataTransfer.files, $this.options);
xhrupload(e.originalEvent.dataTransfer.files, $this.options);
}

}).on("dragenter", function(e){
Expand Down Expand Up @@ -92,9 +92,6 @@
return supportFileAPI() && supportAjaxUploadProgressEvents() && supportFormData();
})();

if (UI.support.ajaxupload){
UI.$.event.props.push("dataTransfer");
}

function xhrupload(files, settings) {

Expand Down

0 comments on commit 02d5f2c

Please sign in to comment.