google / blockly

The web-based visual programming editor.
https://developers.google.com/blockly/
Apache License 2.0
12.51k stars 3.72k forks source link

refactor: remove redundant flyout positioning. #8573

Closed gonfunko closed 2 months ago

gonfunko commented 2 months ago

The basics

The details

This PR removes a redundant call to Flyout.position() in WorkspaceSvg. The code calls Toolbox.position() on the previous line, which in turn calls Flyout.position(), so the flyout was needlessly being positioned twice.