fixes #10134 – dont use jquery offset directly because it uses sub pi…

…xel rendering
twbs · Dec 24, 2013 · a93a753 · a93a753
1 parent fe7fffe
commit a93a753
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 8 deletions.
diff --git a/dist/js/bootstrap.js b/dist/js/bootstrap.js
@@ -1201,9 +1201,18 @@ if (typeof jQuery === "undefined") { throw new Error("Bootstrap requires jQuery"
     offset.top  = offset.top  + marginTop
     offset.left = offset.left + marginLeft
 
-    $tip
-      .offset(offset)
-      .addClass('in')
+    // $.fn.offset doesn't round pixel values
+    // so we use setOffset directly with our own function B-0
+    jQuery.offset.setOffset($tip[0], $.extend({
+      using: function (props) {
+        $tip.css({
+          top: Math.round(props.top),
+          left: Math.round(props.left)
+        })
+      }
+    }, offset), 0)
+
+    $tip.addClass('in')
 
     // check to see if placing tip in new offset caused the tip to resize itself
     var actualWidth  = $tip[0].offsetWidth