From 75655e5758bc786989f26a98b09aabbfb88f66fd Mon Sep 17 00:00:00 2001
From: Sylvester Keil <sylvester@keil.or.at>
Date: Sat, 15 Jan 2011 13:56:20 +0100
Subject: [PATCH] Use parseFloat instead of parseInt to read CSS values.

This fixes #7730 and #7885.
---
 src/offset.js       |  8 +++++---
 test/unit/offset.js | 26 ++++++++++++++++++++++++++
 2 files changed, 31 insertions(+), 3 deletions(-)

diff --git a/src/offset.js b/src/offset.js
index 2040c9d83a..94b67c77a9 100644
--- a/src/offset.js
+++ b/src/offset.js
@@ -187,11 +187,13 @@ jQuery.offset = {
 		// need to be able to calculate position if either top or left is auto and position is absolute
 		if ( calculatePosition ) {
 			curPosition = curElem.position();
+			curTop = curPosition.top;
+			curLeft = curPosition.left;
+		} else {
+			curTop = parseFloat( curCSSTop ) || 0;
+			curLeft = parseFloat( curCSSLeft ) || 0;
 		}
 
-		curTop  = calculatePosition ? curPosition.top  : parseInt( curCSSTop,  10 ) || 0;
-		curLeft = calculatePosition ? curPosition.left : parseInt( curCSSLeft, 10 ) || 0;
-
 		if ( jQuery.isFunction( options ) ) {
 			options = options.call( elem, i, curOffset );
 		}
diff --git a/test/unit/offset.js b/test/unit/offset.js
index cfa14449b7..1f8c3b15c9 100644
--- a/test/unit/offset.js
+++ b/test/unit/offset.js
@@ -422,6 +422,32 @@ test("offsetParent", function(){
 	equals( div[1], jQuery("#nothiddendiv")[0], "The div is the offsetParent." );
 });
 
+test("fractions (see #7730 and #7885)", function() {
+	expect(2);
+	
+	jQuery('body').append('<div id="fractions"/>');
+	
+	var expected = { top: 1000, left: 1000 };
+	var div = jQuery('#fractions');
+	
+	div.css({
+		position: 'absolute',
+		left: '1000.7432222px',
+		top: '1000.532325px',
+		width: 100,
+		height: 100
+	});
+	
+	div.offset(expected);
+	
+	var result = div.offset();
+
+	equals( result.top, expected.top, "Check top" );
+	equals( result.left, expected.left, "Check left" );
+	
+	div.remove();
+});
+
 function testoffset(name, fn) {
 
 	test(name, function() {