jquery / jquery-ui

Source
Commits
Network (55)
Graphs
Tree: dc94bbf

The official jQuery user interface library. — Read more

This URL has Read+Write access

Draggable: Don't use computed style for top and left of position:relative 
elements. Fixes #5537 - Draggable: position relative draggable jumps on first 
drag in Opera

rdworth (author)

Thu Apr 22 06:56:35 -0700 2010

commit dc94bbf53628ada1b5d8
tree 8011bd7ff35e674a5bf5
parent 49f8c1c60326f4b98592

ui/jquery.ui.draggable.js

4 ••••

ui/jquery.ui.draggable.js

View file @ dc94bbf

@@ -305,8 +305,8 @@ $.widget("ui.draggable", $.ui.mouse, {
     if(this.cssPosition == "relative") {
       var p = this.element.position();
       return {
-        top: p.top - (parseInt(this.helper.css("top"),10) || 0) + this.scrollParent.scrollTop(),
-        left: p.left - (parseInt(this.helper.css("left"),10) || 0) + this.scrollParent.scrollLeft()
+        top: p.top - (parseInt(this.helper[0].style.top,10) || 0) + this.scrollParent.scrollTop(),
+        left: p.left - (parseInt(this.helper[0].style.left,10) || 0) + this.scrollParent.scrollLeft()
       };
     } else {
       return { top: 0, left: 0 };

0 notes on commit `dc94bbf` Show line notes below

Please log in to comment.

...	...	@@ -305,8 +305,8 @@ $.widget("ui.draggable", $.ui.mouse, {
305	305	if(this.cssPosition == "relative") {
306	306	var p = this.element.position();
307	307	return {
308		- top: p.top - (parseInt(this.helper.css("top"),10) \|\| 0) + this.scrollParent.scrollTop(),
309		- left: p.left - (parseInt(this.helper.css("left"),10) \|\| 0) + this.scrollParent.scrollLeft()
	308	+ top: p.top - (parseInt(this.helper[0].style.top,10) \|\| 0) + this.scrollParent.scrollTop(),
	309	+ left: p.left - (parseInt(this.helper[0].style.left,10) \|\| 0) + this.scrollParent.scrollLeft()
310	310	};
311	311	} else {
312	312	return { top: 0, left: 0 };

jquery / jquery-ui

0 notes on commit dc94bbf Show line notes below

0 notes on commit `dc94bbf` Show line notes below