0.1.24

* Fix IE11 for in bug in motor field

* minor fix

editor/field_motors.ts

@@ -406,7 +406,7 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
         let opts = {};
         let conts = {};
         let vals = {};
-        for (let opt in options) {
+        for (let opt = 0; opt < options.length; opt++) {
             let text = options[opt][0].alt ? options[opt][0].alt : options[opt][0];
             if (text.indexOf(' ') == -1) {
                 // Patch dual motors as they don't have prefixes.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "pxt-ev3",
-  "version": "0.1.22",
+  "version": "0.1.24",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pxt-ev3",
-  "version": "0.1.22",
+  "version": "0.1.24",
   "description": "LEGO MINDSTORMS EV3 for Microsoft MakeCode",
   "private": true,
   "keywords": [
@@ -46,7 +46,7 @@
   },
   "dependencies": {
     "pxt-common-packages": "0.20.38",
-    "pxt-core": "3.8.13"
+    "pxt-core": "3.8.14"
   },
   "scripts": {
     "test": "node node_modules/pxt-core/built/pxt.js travis"

sim/visuals/controls/colorWheel.ts

@@ -54,7 +54,8 @@ namespace pxsim.visuals {
             this.group = svg.elt("g") as SVGGElement;
 
             let gc = "gradient-color-" + this.getPort();
-            this.colorGradient = svg.linearGradient(globalDefs, gc, false);
+            const prevColorGradient = globalDefs.querySelector(`#${gc}`) as SVGLinearGradientElement;
+            this.colorGradient = prevColorGradient ? prevColorGradient : svg.linearGradient(globalDefs, gc, false);
             svg.setGradientValue(this.colorGradient, "50%");
             svg.setGradientColors(this.colorGradient, "black", "yellow");
 
@@ -99,10 +100,5 @@ namespace pxsim.visuals {
 
             return this.group;
         }
-
-        public dispose() {
-            if (this.colorGradient) this.colorGradient.parentElement.removeChild(this.colorGradient);
-            super.dispose();
-        }
     }
 }

sim/visuals/controls/distanceSlider.ts

@@ -15,7 +15,8 @@ namespace pxsim.visuals {
         getInnerView(parent: SVGSVGElement, globalDefs: SVGDefsElement) {
             let gid = "gradient-slider-" + this.getPort();
             this.group = svg.elt("g") as SVGGElement;
-            this.gradient = createGradient(gid, this.getGradientDefinition());
+            const prevGradient = globalDefs.querySelector(`#${gid}`) as SVGLinearGradientElement;
+            this.gradient = prevGradient ? prevGradient : createGradient(gid, this.getGradientDefinition());
             this.gradient.setAttribute('x1', '0%');
             this.gradient.setAttribute('y1', '0%');
             this.gradient.setAttribute('x2', '0%');
@@ -74,11 +75,6 @@ namespace pxsim.visuals {
             return this.group;
         }
 
-        public dispose() {
-            if (this.gradient) this.gradient.parentElement.removeChild(this.gradient);
-            super.dispose();
-        }
-
         getInnerHeight() {
             return 192;
         }