From 8d3dea4c741f3fa7cdaff5a43c0a8e0c120e4194 Mon Sep 17 00:00:00 2001
From: DD <liudi08@gmail.com>
Date: Wed, 8 Aug 2018 20:44:23 -0400
Subject: [PATCH] Add issue to todo and getter for isVisible on workspace

---
 core/blockly.js       |  6 +++---
 core/workspace_svg.js | 17 +++++++++++++----
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/core/blockly.js b/core/blockly.js
index 974e39d11..81c0a7377 100644
--- a/core/blockly.js
+++ b/core/blockly.js
@@ -172,11 +172,11 @@ Blockly.svgResize = function(workspace) {
  * @param {!Event} e Key down event.
  * @private
  */
- // TODO handle cases where there are multiple workspaces and non-main workspaces are able to accept
- // input.
+ // TODO (https://github.com/google/blockly/issues/1998) handle cases where there are multiple workspaces
+ // and non-main workspaces are able to accept input.
 Blockly.onKeyDown_ = function(e) {
   if (Blockly.mainWorkspace.options.readOnly || Blockly.utils.isTargetInput(e)
-      || (Blockly.mainWorkspace.rendered && !Blockly.mainWorkspace.isVisible)) {
+      || (Blockly.mainWorkspace.rendered && !Blockly.mainWorkspace.isVisible())) {
     // No key actions on readonly workspaces.
     // When focused on an HTML text input widget, don't trap any keys.
     // Ignore keypresses on rendered workspaces that have been explicitly
diff --git a/core/workspace_svg.js b/core/workspace_svg.js
index 6ab972d44..ce6e91174 100644
--- a/core/workspace_svg.js
+++ b/core/workspace_svg.js
@@ -139,11 +139,11 @@ Blockly.WorkspaceSvg.prototype.rendered = true;
 
 /**
  * Whether the workspace is visible.  False if the workspace has been hidden
- * by calling `setVisisble(false)`.
+ * by calling `setVisible(false)`.
  * @type {boolean}
- * @package
+ * @private
  */
-Blockly.WorkspaceSvg.prototype.isVisible = true;
+Blockly.WorkspaceSvg.prototype.isVisible_ = true;
 
 /**
  * Is this workspace the surface for a flyout?
@@ -342,6 +342,15 @@ Blockly.WorkspaceSvg.prototype.updateInverseScreenCTM = function() {
   this.inverseScreenCTMDirty_ = true;
 };
 
+/**
+ * Getter for isVisible
+ * @return {boolean} Whether the workspace is visible.  False if the workspace has been hidden
+ * by calling `setVisible(false)`.
+ */
+Blockly.WorkspaceSvg.prototype.isVisible = function() {
+  return this.isVisible_;
+};
+
 /**
  * Return the absolute coordinates of the top-left corner of this element,
  * scales that after canvas SVG element, if it's a descendant.
@@ -890,7 +899,7 @@ Blockly.WorkspaceSvg.prototype.setVisible = function(isVisible) {
   } else {
     Blockly.hideChaff(true);
   }
-  this.isVisible = isVisible;
+  this.isVisible_ = isVisible;
 };
 
 /**