Simplified implementation and fixed smaller issues.

Author: MarkusBordihn

build/externs/blockly.js

@@ -230,7 +230,7 @@ Blockly.WorkspaceSvg.prototype.setScale = function(newScale) {};
 Blockly.WorkspaceSvg.prototype.scrollCenter = function() {};
 
 /**
- * @param {Node|string} tree
+ * @param {Node|string=} tree
  */
 Blockly.WorkspaceSvg.prototype.updateToolbox = function(tree) {};
 

locales/deu/whats_new_screen.js

@@ -0,0 +1,23 @@
+/**
+ * @fileoverview Translation file for the What's New screen (german).
+ *
+ * @license Copyright 2018 The Coding with Chrome Authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * @author [email protected] (Markus Bordihn)
+ */
+/* eslint-disable max-len */
+Locales['deu']['WHATS_NEW_SCREEN'] = {
+  'SHOW_ON_CHANGE': 'Zeige diese Benachrichtigung, wenn eine neue Version installiert ist.',
+};

locales/eng/select_screen.js

@@ -24,7 +24,6 @@ Locales['eng']['SELECT_SCREEN'] = {
   'BLUETOOTH_WEB_NOT_AVAILABLE': 'Required Web Bluetooth support is not available.',
   'LOAD_FILE': 'Load {$title}',
   'USE': 'Use',
-  'ENABLE_WHATS_NEW': 'Show notification when a new version is installed.',
 };
 
 

locales/eng/whats_new_screen.js

@@ -0,0 +1,23 @@
+/**
+ * @fileoverview Translation file for the What's New screen (english).
+ *
+ * @license Copyright 2018 The Coding with Chrome Authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * @author [email protected] (Markus Bordihn)
+ */
+/* eslint-disable max-len */
+Locales['eng']['WHATS_NEW_SCREEN'] = {
+  'SHOW_ON_CHANGE': 'Show notification when a new version is installed.',
+};

src/addon/message/message.js

@@ -23,6 +23,7 @@ goog.require('cwc.mode.Modder.Events');
 goog.require('cwc.mode.Type');
 goog.require('cwc.soy.addon.Message');
 goog.require('cwc.ui.SelectScreen.Events');
+goog.require('cwc.utils.Events');
 goog.require('cwc.utils.Logger');
 
 

src/frameworks/lego/ev3/ev3.js

@@ -33,8 +33,7 @@ goog.require('cwc.protocol.lego.ev3.Robots');
 
 /**
  * @constructor
- * @struct
- * @final
+ * @unrestricted
  * @export
  */
 cwc.framework.lego.Ev3 = function() {

src/frameworks/messenger/command.js

@@ -21,7 +21,7 @@ goog.provide('cwc.framework.MessengerCommand');
 
 
 /**
- * @param {!cwc.Messenger} messenger
+ * @param {!cwc.framework.Messenger} messenger
  * @constructor
  * @struct
  * @final
@@ -30,7 +30,7 @@ cwc.framework.MessengerCommand = function(messenger) {
   /** @type {string} */
   this.name = 'Messenger Display';
 
-  /** @private {!cwc.Messenger} */
+  /** @private {!cwc.framework.Messenger} */
   this.messenger_ = messenger;
 
   // Global mapping

src/frameworks/messenger/display.js

@@ -21,7 +21,7 @@ goog.provide('cwc.framework.MessengerDisplay');
 
 
 /**
- * @param {!Function} messenger
+ * @param {!cwc.framework.Messenger} messenger
  * @constructor
  * @struct
  * @final
@@ -30,7 +30,7 @@ cwc.framework.MessengerDisplay = function(messenger) {
   /** @type {string} */
   this.name = 'Messenger Display';
 
-  /** @private {!cwc.Messenger} */
+  /** @private {!cwc.framework.Messenger} */
   this.messenger_ = messenger;
 
   // Global mapping

src/frameworks/sphero/sprk_plus/sprk_plus.js

@@ -57,7 +57,7 @@ cwc.framework.sphero.SprkPlus = function() {
   /** @private {number} */
   this.heading_ = 0;
 
-  /** @private {boolean} */
+  /** @private {boolean|undefined} */
   this.state_ = undefined;
 
   /** @private {!cwc.framework.Messenger} */
@@ -140,8 +140,8 @@ cwc.framework.sphero.SprkPlus.prototype.setMotionTimeout = function(timeout,
  * @param {number=} delay in msec
  * @export
  */
-cwc.framework.sphero.SprkPlus.prototype.roll = function(speed, heading, state,
-    delay) {
+cwc.framework.sphero.SprkPlus.prototype.roll = function(speed, heading = 0,
+    state, delay) {
   this.heading_ = heading;
   this.state_ = state;
   this.messenger_.send('roll', {
@@ -260,7 +260,7 @@ cwc.framework.sphero.SprkPlus.prototype.handleVelocity_ = function(data) {
 
 
 /**
- * @param {event} e
+ * @param {Event} e
  * @private
  */
 cwc.framework.sphero.SprkPlus.prototype.handlePositionChange_ = function(e) {

src/ui/builder.js

@@ -271,6 +271,7 @@ cwc.ui.Builder.prototype.decorateUI = function() {
       this.setProgressFunc('Prepare GAPI ...', this.prepareGAPI);
     }
     this.setProgressFunc('Loading select screen ...', this.showSelectScreen);
+    this.setProgressFunc('Loading whats new screen ...', this.showWhatsNew);
     this.setProgressFunc('Loading cache ...', this.loadCache).then(() => {
       // Done.
       this.setProgress('Starting Coding with Chrome', 100);
@@ -571,6 +572,18 @@ cwc.ui.Builder.prototype.showSelectScreen = function() {
   }
 };
 
+
+/**
+ * Shows what's new screen.
+ */
+cwc.ui.Builder.prototype.showWhatsNew = function() {
+  let whatsNewScreen = this.helper.getInstance('whatsNew');
+  if (whatsNewScreen) {
+    whatsNewScreen.showWhatsNewScreen();
+  }
+};
+
+
 /**
  * @param {string} name
  * @private