fix: Fix downloading screenshots on the playground. (#6025)

* fix: Fix downloading screenshots on the playground. * fix: Resolve lint warning about unused variable.
google · Mar 25, 2022 · ca6e590 · ca6e590
1 parent 2c15d00
commit ca6e590
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/tests/playground.html b/tests/playground.html
@@ -7,6 +7,7 @@
   <!-- This script loads uncompressed when on localhost and loads compressed when it is being hosted.
        Please add any other dependencies to playgrounds/prepare.js.-->
 <script src="playgrounds/prepare.js"></script>
+<script src="playgrounds/screenshot.js"></script>
 <script src="../node_modules/@blockly/dev-tools/dist/index.js"></script>
 <script type=module>
 import Blockly from './playgrounds/blockly.mjs';
@@ -242,7 +243,7 @@
     text: 'Download Screenshot',
     enabled: workspace.getTopBlocks().length,
     callback: function() {
-      Blockly.downloadScreenshot(workspace);
+      downloadScreenshot(workspace);
     }
   };
   menuOptions.push(screenshotOption);

diff --git a/tests/playgrounds/advanced_playground.html b/tests/playgrounds/advanced_playground.html
@@ -121,7 +121,7 @@
     text: 'Download Screenshot',
     enabled: workspace.getTopBlocks().length,
     callback: function() {
-      Blockly.downloadScreenshot(workspace);
+      downloadScreenshot(workspace);
     }
   };
   menuOptions.push(screenshotOption);

diff --git a/tests/playgrounds/screenshot.js b/tests/playgrounds/screenshot.js
@@ -95,7 +95,7 @@ function workspaceToSvg_(workspace, callback, customCss) {
  * Download a screenshot of the blocks on a Blockly workspace.
  * @param {!Blockly.WorkspaceSvg} workspace The Blockly workspace.
  */
-Blockly.downloadScreenshot = function(workspace) {
+function downloadScreenshot(workspace) {
   workspaceToSvg_(workspace, function(datauri) {
     const a = document.createElement('a');
     a.download = 'screenshot.png';