Fix scrollbar visibility when horizontal

rachel-fenichel · rachel-fenichel · commit 6ec77a597a21 · 2016-01-27T08:20:26.000-08:00
diff --git a/core/scrollbar.js b/core/scrollbar.js
@@ -243,7 +243,7 @@ Blockly.Scrollbar.prototype.resize = function(opt_metrics) {
       // Only show the scrollbar if needed.
       // Ideally this would also apply to scrollbar pairs, but that's a bigger
       // headache (due to interactions with the corner square).
-      this.setVisible(outerLength < hostMetrics.contentHeight);
+      this.setVisible(outerLength < hostMetrics.contentWidth);
     }
     this.ratio_ = outerLength / hostMetrics.contentWidth;
     if (this.ratio_ === -Infinity || this.ratio_ === Infinity ||

Original file line number	Diff line number	Diff line change
`@@ -243,7 +243,7 @@ Blockly.Scrollbar.prototype.resize = function(opt_metrics) {`
`243`	`243`	`// Only show the scrollbar if needed.`
`244`	`244`	`// Ideally this would also apply to scrollbar pairs, but that's a bigger`
`245`	`245`	`// headache (due to interactions with the corner square).`
`246`		`- this.setVisible(outerLength < hostMetrics.contentHeight);`
	`246`	`+ this.setVisible(outerLength < hostMetrics.contentWidth);`
`247`	`247`	`}`
`248`	`248`	`this.ratio_ = outerLength / hostMetrics.contentWidth;`
`249`	`249`	`if (this.ratio_ === -Infinity \|\| this.ratio_ === Infinity \|\|`