Use bounds in memory, rather than accessing DOM

Aaron Imming · Aaron Imming · commit 7501d4f20fd4 · 2019-07-05T14:58:51.000+08:00
diff --git a/src/scratch/hooks/flyout_vertical.js b/src/scratch/hooks/flyout_vertical.js
@@ -162,7 +162,7 @@ Blockly.VerticalFlyout.prototype.position = function() {
         if (targetWorkspaceMetrics.toolboxWidth) {
             // deriv-bot: Allow for dynamic toolbox width, specifies where to position
             // the flyout.
-            x = this.parentToolbox_.HtmlDiv.clientWidth + Blockly.BlockSvg.TAB_WIDTH;
+            x = this.toolbox_bounds_.width + Blockly.BlockSvg.TAB_WIDTH;
         }
     } else {
         // Because the anchor point of the flyout is on the left, but we want

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ Blockly.VerticalFlyout.prototype.position = function() {`
`162`	`162`	`if (targetWorkspaceMetrics.toolboxWidth) {`
`163`	`163`	`// deriv-bot: Allow for dynamic toolbox width, specifies where to position`
`164`	`164`	`// the flyout.`
`165`		`- x = this.parentToolbox_.HtmlDiv.clientWidth + Blockly.BlockSvg.TAB_WIDTH;`
	`165`	`+ x = this.toolbox_bounds_.width + Blockly.BlockSvg.TAB_WIDTH;`
`166`	`166`	`}`
`167`	`167`	`} else {`
`168`	`168`	`// Because the anchor point of the flyout is on the left, but we want`