Potential fix for IE11 scrollbar issue. fixes jackmoore#327, fixes ja…

…ckmoore#226
4zik88 · Apr 26, 2017 · 29ca460 · 29ca460
1 parent 6c198a8
commit 29ca460
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/dist/autosize.js b/dist/autosize.js
@@ -156,22 +156,24 @@
 
 			var styleHeight = Math.round(parseFloat(ta.style.height));
 			var computed = window.getComputedStyle(ta, null);
-			var actualHeight = Math.round(parseFloat(computed.height));
+
+			// Using offsetHeight as a replacement for computed.height in IE, because IE does not account use of border-box
+			var actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(computed.height)) : ta.offsetHeight;
 
 			// The actual height not matching the style height (set via the resize method) indicates that
 			// the max-height has been exceeded, in which case the overflow should be set to visible.
 			if (actualHeight !== styleHeight) {
 				if (computed.overflowY !== 'visible') {
 					changeOverflow('visible');
 					resize();
-					actualHeight = Math.round(parseFloat(window.getComputedStyle(ta, null).height));
+					actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(window.getComputedStyle(ta, null).height)) : ta.offsetHeight;
 				}
 			} else {
 				// Normally keep overflow set to hidden, to avoid flash of scrollbar as the textarea expands.
 				if (computed.overflowY !== 'hidden') {
 					changeOverflow('hidden');
 					resize();
-					actualHeight = Math.round(parseFloat(window.getComputedStyle(ta, null).height));
+					actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(window.getComputedStyle(ta, null).height)) : ta.offsetHeight;
 				}
 			}
 

diff --git a/dist/autosize.min.js b/dist/autosize.min.js
diff --git a/src/autosize.js b/src/autosize.js
@@ -134,22 +134,24 @@ function assign(ta) {
 
 		const styleHeight = Math.round(parseFloat(ta.style.height));
 		const computed = window.getComputedStyle(ta, null);
-		var actualHeight = Math.round(parseFloat(computed.height));
+
+		// Using offsetHeight as a replacement for computed.height in IE, because IE does not account use of border-box
+		var actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(computed.height)) : ta.offsetHeight;
 
 		// The actual height not matching the style height (set via the resize method) indicates that 
 		// the max-height has been exceeded, in which case the overflow should be set to visible.
 		if (actualHeight !== styleHeight) {
 			if (computed.overflowY !== 'visible') {
 				changeOverflow('visible');
 				resize();
-				actualHeight = Math.round(parseFloat(window.getComputedStyle(ta, null).height));
+				actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(window.getComputedStyle(ta, null).height)) : ta.offsetHeight;
 			}
 		} else {
 			// Normally keep overflow set to hidden, to avoid flash of scrollbar as the textarea expands.
 			if (computed.overflowY !== 'hidden') {
 				changeOverflow('hidden');
 				resize();
-				actualHeight = Math.round(parseFloat(window.getComputedStyle(ta, null).height));
+				actualHeight = computed.boxSizing === 'content-box' ? Math.round(parseFloat(window.getComputedStyle(ta, null).height)) : ta.offsetHeight;
 			}
 		}