fix: fix bug when scrollHeight is slightly higher than clientHeight

clampy-js · May 14, 2018 · 225c5a1 · 225c5a1
1 parent 7ce42e3
commit 225c5a1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/clampy.ts b/src/clampy.ts
@@ -130,9 +130,9 @@ export function clamp(element: HTMLElement, options?: ClampOptions): ClampRespon
    * Note: inline elements return 0 for scrollHeight and clientHeight
    */
   function getElemHeight(elem: HTMLElement): number {
-    // The '- 1' is a hack to deal with the element height when the browser(especially IE) zoom level is not 100%.
+    // The '- 4' is a hack to deal with the element height when the browser(especially IE) zoom level is not 100%.
     // It also doesn't impact clamping when the browser zoom level is 100%.
-    return Math.max(elem.scrollHeight, elem.clientHeight) - 1
+    return Math.max(elem.scrollHeight, elem.clientHeight) - 4
   }
 
   /**