Fixes #413 (again). No position:absolute needed when not changing sty…

…les anyway.
mootools · Oct 15, 2010 · 48c6b2b · 48c6b2b
1 parent e244fcf
commit 48c6b2b
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/Source/Drag/Drag.Move.js b/Source/Drag/Drag.Move.js
@@ -50,16 +50,18 @@ Drag.Move = new Class({
 		if (this.container && typeOf(this.container) != 'element')
 			this.container = document.id(this.container.getDocument().body);
 
-		if (this.options.style && this.options.modifiers.x == "left" && this.options.modifiers.y == "top"){
-			var parentStyles,
-				parent = element.getOffsetParent();
-			var styles = element.getStyles('left', 'top');
-			if (parent && styles.left == 'auto' || styles.top == 'auto'){
-				element.setPosition(element.getPosition(parent));
+		if (this.options.style){
+			if (this.options.modifiers.x == "left" && this.options.modifiers.y == "top"){
+				var parentStyles,
+					parent = element.getOffsetParent();
+				var styles = element.getStyles('left', 'top');
+				if (parent && styles.left == 'auto' || styles.top == 'auto'){
+					element.setPosition(element.getPosition(parent));
+				}
 			}
-		}
 
-		if (element.getStyle('position') == 'static') element.setStyle('position', 'absolute');
+			if (element.getStyle('position') == 'static') element.setStyle('position', 'absolute');
+		}
 
 		this.addEvent('start', this.checkDroppables, true);
 		this.overed = null;

diff --git a/Tests/Drag/Drag.Move_(style_false).html b/Tests/Drag/Drag.Move_(style_false).html
@@ -6,7 +6,6 @@
 		background-color: #090;
 		text-align: center;
 		line-height: 30px;
-		position: relative;
 	}
 </style>