From 5b00c217190538fc1e03dcd69b1cadd3e66d5dc9 Mon Sep 17 00:00:00 2001 From: Richard Worth Date: Sat, 28 Feb 2009 20:45:05 +0000 Subject: [PATCH] slider: use handle width / 2 instead of marginLeft to find offset of mousedown relative to center of handle (fixes #4063 - slider: horizontal slider handle jumps up when you click on it IE6) --- ui/ui.slider.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/ui.slider.js b/ui/ui.slider.js index c651044b2..8f658abed 100644 --- a/ui/ui.slider.js +++ b/ui/ui.slider.js @@ -222,7 +222,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { var offset = closestHandle.offset(); var mouseOverHandle = !$(event.target).parents().andSelf().is('.ui-slider-handle'); this._clickOffset = mouseOverHandle ? { left: 0, top: 0 } : { - left: event.pageX - offset.left + (parseInt(closestHandle.css('marginLeft'),10) || 0), + left: event.pageX - offset.left - (closestHandle.width() / 2), top: event.pageY - offset.top - (closestHandle.height() / 2) - (parseInt(closestHandle.css('borderTopWidth'),10) || 0)