Skip to content
Permalink
Browse files

simulate - drag: added mousemove [+1, +1] after mousedown. Removed mo…

…useover and mouseout as they're not minimal/required for drag. They were there as work-around for resizable. Moved work-around to resizable test.
  • Loading branch information
rdworth committed Jun 21, 2008
1 parent 2e920f0 commit 2361bda9bae169c92fb05abc935b834cbb119919
Showing with 6 additions and 2 deletions.
  1. +4 −0 tests/resizable.js
  2. +2 −2 tests/simulate/jquery.simulate.js
@@ -8,6 +8,10 @@ var drag = function(el, dx, dy, complete) {
// speed = sync -> Drag syncrhonously.
// speed = fast|slow -> Drag asyncrhonously - animated.

//this mouseover is to work around a limitation in resizable
//TODO: fix resizable so handle doesn't require mouseover in order to be used
$(el).simulate("mouseover");

return $(el).simulate("drag", {
dx: dx||0, dy: dy||0, speed: 'sync', complete: complete
});
@@ -115,13 +115,13 @@ $.extend($.simulate.prototype, {
options = this.options, x = Math.floor(center.x), y = Math.floor(center.y),
dx = options.dx || 0, dy = options.dy || 0, target = this.target;
var coord = { clientX: x, clientY: y };
this.simulateEvent(target, "mouseover");
this.simulateEvent(target, "mousedown", coord);
coord = { clientX: x + 1, clientY: y + 1 };
this.simulateEvent(document, "mousemove", coord);
coord = { clientX: x + dx, clientY: y + dy };
this.simulateEvent(document, "mousemove", coord);
this.simulateEvent(document, "mousemove", coord);
this.simulateEvent(target, "mouseup", coord);
this.simulateEvent(target, "mouseout");
},
findCenter: function(el) {
var el = $(this.target), o = el.offset();

0 comments on commit 2361bda

Please sign in to comment.
You can’t perform that action at this time.