fix(columnWidth): Compensate for scrollbar only if there is vertical …

…scroll
frappe · Aug 23, 2018 · a04a5a1 · a04a5a1
1 parent 7531e13
commit a04a5a1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/dom.js b/src/dom.js
@@ -200,3 +200,7 @@ $.scrollbarWidth = function scrollbarWidth() {
 
     return scrollbarWidth;
 };
+
+$.hasVerticalOverflow = function (element) {
+    return element.scrollHeight > element.offsetHeight + 10;
+};
diff --git a/src/style.js b/src/style.js
@@ -206,6 +206,8 @@ export default class Style {
     }
 
     compensateScrollbarWidth() {
+        if (!$.hasVerticalOverflow(this.bodyScrollable)) return;
+
         requestAnimationFrame(() => {
             const scrollbarWidth = $.scrollbarWidth();
             const lastCol = this.datamanager.getColumn(-1);