From 2361bda9bae169c92fb05abc935b834cbb119919 Mon Sep 17 00:00:00 2001 From: Richard Worth Date: Sat, 21 Jun 2008 10:52:58 +0000 Subject: [PATCH] simulate - drag: added mousemove [+1, +1] after mousedown. Removed mouseover and mouseout as they're not minimal/required for drag. They were there as work-around for resizable. Moved work-around to resizable test. --- tests/resizable.js | 4 ++++ tests/simulate/jquery.simulate.js | 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/resizable.js b/tests/resizable.js index 2e7da9adc..d23cbd7b2 100644 --- a/tests/resizable.js +++ b/tests/resizable.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 }); diff --git a/tests/simulate/jquery.simulate.js b/tests/simulate/jquery.simulate.js index a0dc290da..b55687388 100644 --- a/tests/simulate/jquery.simulate.js +++ b/tests/simulate/jquery.simulate.js @@ -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();