Skip to content

Commit 91ca657

Browse files
committed
feat(draggable): Don't start new drag if previous one hasn't stopped
Moving 4e6159f by @atesgoral into draggable.js
1 parent 4b1b2b7 commit 91ca657

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

src/jquery.draggable.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -246,8 +246,10 @@
246246

247247
fn.on_dragstart = function(e) {
248248
e.preventDefault();
249-
this.drag_start = true;
250-
this.is_dragging = true;
249+
250+
if (this.is_dragging) { return this; }
251+
252+
this.drag_start = this.is_dragging = true;
251253
var offset = this.$container.offset();
252254
this.baseX = Math.round(offset.left);
253255
this.baseY = Math.round(offset.top);

src/jquery.gridster.js

-8
Original file line numberDiff line numberDiff line change
@@ -661,13 +661,6 @@
661661
offset_left: this.options.widget_margins[0],
662662
container_width: this.container_width,
663663
start: function(event, ui) {
664-
// Ignore drag start if mouse was released outside screen on a previous drag
665-
if (self.dragging) {
666-
return;
667-
}
668-
669-
self.dragging = true;
670-
671664
self.$widgets.filter('.player-revert')
672665
.removeClass('player-revert');
673666

@@ -680,7 +673,6 @@
680673
self.$el.trigger('gridster:dragstart');
681674
},
682675
stop: function(event, ui) {
683-
self.dragging = false;
684676
self.on_stop_drag.call(self, event, ui);
685677
self.$el.trigger('gridster:dragstop');
686678
},

0 commit comments

Comments
 (0)