1.0.8

* Use the value for options, and hardcode localized versions of first dropdown

docs/SUMMARY.md

@@ -82,12 +82,12 @@
             * [reset](/reference/sensors/gyro/reset)
         * [Ultrasonic](/reference/sensors/ultrasonic)
             * [on event](/reference/sensors/ultrasonic/on-event)
-            * [distance](reference/sensors/ultrasonic/distance)
-            * [pause until](reference/sensors/ultrasonic/pause-until)
+            * [distance](/reference/sensors/ultrasonic/distance)
+            * [pause until](/reference/sensors/ultrasonic/pause-until)
         * [Infrared](/reference/sensors/infrared)
             * [on event](/reference/sensors/infrared/on-event)
-            * [distance](reference/sensors/infrared/proximity)
-            * [pause until](reference/sensors/infrared/pause-until)
+            * [distance](/reference/sensors/infrared/proximity)
+            * [pause until](/reference/sensors/infrared/pause-until)
         * [Infrared beacon](/reference/sensors/beacon)
             * [on event](/reference/sensors/beacon/on-event)
             * [pause until](/reference/sensors/beacon/pause-until)

docs/design-engineering/make-it-smarter.md

@@ -36,22 +36,16 @@ Think about a creature’s movement for inspiration. Your mechanism can be attac
 
 More building ideas:
 
-| | | | | |
-|-|-|-|-|-|
-|[![EV3 Frames](/static/lessons/make-it-smarter/ev3-parts-frames.jpg)][EV3 Frames] | |[![EV3 Color Sensor 1](/static/lessons/make-it-smarter/ev3-parts-color-sensor-1.jpg)][Color Sensor 1] | |[![EV3 Gyro Sensor](/static/lessons/make-it-smarter/ev3-parts-gyro-sensor.jpg)][Gyro Sensor] |
-| [EV3 Frames] | | [Color Sensor 1] | | [Gyro Sensor] |
-<br/>
 
-| | | | | |
-|-|-|-|-|-|
-|[![EV3 Ultrasonic Sensor](/static/lessons/make-it-smarter/ev3-parts-ultrasonic-sensor.jpg)][Ultrasonic Sensor] | | [![EV3 Touch Sensor](/static/lessons/make-it-smarter/ev3-parts-touch-sensor.jpg)][Touch Sensor] | | [![EV3 Gyro Sensor](/static/lessons/make-it-smarter/ev3-parts-jaw.jpg)][Jaw] |
-| [Ultrasonic Sensor] | | [Touch Sensor] | | [Jaw] |
-<br/>
-
-| | | | | |
-|-|-|-|-|-|
-| [![EV3 Leg 1](/static/lessons/make-it-smarter/ev3-parts-leg-1.jpg)][Leg 1] | | [![EV3 Leg 2](/static/lessons/make-it-smarter/ev3-parts-leg-2.jpg)][Leg 2] | | [![EV3 Leg 3](/static/lessons/make-it-smarter/ev3-parts-leg-3.jpg)][Leg 3] |
-| [Leg 1] | | [Leg 2] | | [Leg 3] |
+* [EV3 Frames]
+* [Color Sensor 1]
+* [Gyro Sensor]
+* [Ultrasonic Sensor]
+* [Touch Sensor]
+* [Jaw]
+* [Leg 1] 
+* [Leg 2]
+* [Leg 3]
 
 ### ~hint
 

docs/maker/security-gadget.md

@@ -42,14 +42,12 @@ This activity uses sensor inputs. You may want to try the [Use](/getting-started
 
 Follow the steps of the _Maker Design Process_ for this lesson:
 
-| | |
-|-|-|
-| ![Paper sheet icon](/static/maker/design-process-icons/define-problem.png) | **Define the Problem** |
-| ![Lightbulb icon](/static/maker/design-process-icons/brainstorming.png) | **Brainstorming** |
-| ![Checkmark icon](/static/maker/design-process-icons/define-criteria.png) | **Define the Design Criteria** |
-| ![Connector icon](/static/maker/design-process-icons/go-make.png) | **Go Make** |
-| ![Redo icon](/static/maker/design-process-icons/review-revise.png) | **Review and Revise Your Solution** |
-| ![Speaker icon](/static/maker/design-process-icons/communicate.png) | **Communicate Your Solution** |
+* Define the Problem
+* Brainstorming
+* Define the Design Criteria
+* Go Make
+* Review and Revise Your Solution
+* Communicate Your Solution
 
 ### Defining the Problem
 

docs/maker/sound-machine.md

@@ -41,14 +41,12 @@ This activity uses motor rotations and sensor inputs. You may want to try the [U
 
 Follow the steps of the _Maker Design Process_ for this lesson:
 
-| | |
-|-|-|
-| ![Paper sheet icon](/static/maker/design-process-icons/define-problem.png) | **Define the Problem** |
-| ![Lightbulb icon](/static/maker/design-process-icons/brainstorming.png) | **Brainstorming** |
-| ![Checkmark icon](/static/maker/design-process-icons/define-criteria.png) | **Define the Design Criteria** |
-| ![Connector icon](/static/maker/design-process-icons/go-make.png) | **Go Make** |
-| ![Redo icon](/static/maker/design-process-icons/review-revise.png) | **Review and Revise Your Solution** |
-| ![Speaker icon](/static/maker/design-process-icons/communicate.png) | **Communicate Your Solution** |
+* Define the Problem
+* Brainstorming
+* Define the Design Criteria
+* Go Make
+* Review and Revise Your Solution
+* Communicate Your Solution
 
 ### Defining the Problem
 

docs/tutorials/run-motors.md

@@ -106,9 +106,9 @@ brick.buttonRight.onEvent(ButtonEvent.Pressed, function () {
 
 For the ``||motors:run||`` blocks that are in the ``||brick:on button left||`` and ``||brick:on button right||`` blocks, use the drop-down menu to select a ``medium motor`` on Port ``D``. 
 
-| | | |
-|-|-|-|
-| ![Select a motor type dropdown](/static/tutorials/run-motors/run-motor-dropdown.png) | | | ![Select a motor port dropdown](/static/tutorials/run-motors/motor-port-dropdown.png) |
+![Select a motor type dropdown](/static/tutorials/run-motors/run-motor-dropdown.png)
+<br/>
+![Select a motor port dropdown](/static/tutorials/run-motors/motor-port-dropdown.png)
 
 ```blocks
 brick.buttonUp.onEvent(ButtonEvent.Pressed, function () {

editor/extension.ts

@@ -30,7 +30,7 @@ pxt.editor.initExtensionsAsync = function (opts: pxt.editor.ExtensionOptions): P
                     <div class="ui header">${lf("First time here?")}</div>
                     <strong style="font-size:small">${lf("You must have version 1.10E or above of the firmware")}</strong>
                     <div style="justify-content: center;display: flex;padding: 1rem;">
-                        <img class="ui image" src="./static/download/firmware.png" style="height:100px;" />
+                        <img class="ui image" src="/static/download/firmware.png" style="height:100px;" />
                     </div>
                     <a href="/troubleshoot" target="_blank">${lf("Check your firmware version here and update if needed")}</a>
                 </div>
@@ -42,7 +42,7 @@ pxt.editor.initExtensionsAsync = function (opts: pxt.editor.ExtensionOptions): P
                                     <div class="column">
                                         <div class="ui">
                                             <div class="image">
-                                                <img class="ui medium rounded image" src="./static/download/connect.svg" style="height:109px;width:261px;margin-bottom:1rem;" />
+                                                <img class="ui medium rounded image" src="/static/download/connect.svg" style="height:109px;width:261px;margin-bottom:1rem;" />
                                             </div>
                                             <div class="content">
                                                 <div class="description">
@@ -57,7 +57,7 @@ pxt.editor.initExtensionsAsync = function (opts: pxt.editor.ExtensionOptions): P
                                     <div class="column">
                                         <div class="ui">
                                             <div class="image">
-                                                <img class="ui medium rounded image" src="./static/download/transfer.svg" style="height:109px;width:261px;margin-bottom:1rem;" />
+                                                <img class="ui medium rounded image" src="/static/download/transfer.svg" style="height:109px;width:261px;margin-bottom:1rem;" />
                                             </div>
                                             <div class="content">
                                                 <div class="description">

fieldeditors/field_motors.ts

@@ -133,14 +133,27 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
         this.setText(text);
     }
 
-    getFirstValue(text: string) {
-        // Get first set of words up until last space
-        return this.normalizeText_(text.substring(0, text.lastIndexOf(' ')));
+    getFirstValue(value: string) {
+        const typeValue = value.indexOf('large') != -1 ? 'large' : 'medium';
+        const portValue = this.getSecondValue(value);
+        const isDual = portValue.length > 1;
+        return `${typeValue} motor${isDual ? 's' : ''}`;
     }
 
-    getSecondValue(text: string) {
-        // Get last word
-        return this.normalizeText_(text.match(/\S*$/)[0]);
+    getSecondValue(value: string) {
+        return (value.indexOf('large') != -1) ?
+            value.substring(value.indexOf('large') + 5) :
+            value.substring(value.indexOf('medium') + 6);
+    }
+
+    getFirstValueI11n(value: string) {
+        const firstValue = this.getFirstValue(value);
+        const motorOptions = {
+            'medium motor': lf('medium motor'),
+            'large motor': lf('large motor'),
+            'large motors': lf('large motors')
+        }
+        return motorOptions[firstValue];
     }
 
     private normalizeText_(text: string) {
@@ -198,8 +211,8 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
         if (text === null) {
             return text;
         }
-        if (text.indexOf(' ') == -1) {
-            text = `large motors ${text}`;
+        if (text.indexOf('|') == -1) {
+            text = this.sourceBlock_.RTL ? `${text}|${lf("large motors")}` : `${lf("large motors")}|${text}`;
         }
         return text;
     }
@@ -296,16 +309,14 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
             goog.dom.removeChildren(/** @type {!Element} */(this.textElement_));
             goog.dom.removeChildren(/** @type {!Element} */(this.textElement2_));
 
-            var text = this.text_;
-            text = this.patchDualMotorText(text);
-
             // First dropdown
-            const textNode1 = document.createTextNode(this.getFirstValue(text));
+            // Use one of the following options, medium motor, large motor or large motors (translated)
+            const textNode1 = document.createTextNode(this.getFirstValueI11n(this.value_));
             this.textElement_.appendChild(textNode1);
 
-            // Second dropdown
+            // Second dropdown, no need to translate. Only port numbers
             if (this.textElement2_) {
-                const textNode2 = document.createTextNode(this.getSecondValue(text));
+                const textNode2 = document.createTextNode(this.getSecondValue(this.value_));
                 this.textElement2_.appendChild(textNode2);
             }
             this.updateWidth();
@@ -402,29 +413,28 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
         contentDiv.setAttribute('aria-haspopup', 'true');
         let options = this.getOptions();
 
-        // Hashmap of options
         let opts = {};
         let conts = {};
         let vals = {};
+        // Go through all option values and split them into groups
         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.
-                text = this.patchDualMotorText(text);
-                if (options[opt][0].alt) options[opt][0].alt = text;
-            }
             const value = options[opt][1];
-            const firstValue = this.getFirstValue(text);
-            const secondValue = this.getSecondValue(text);
-            if (!opts[firstValue]) opts[firstValue] = [secondValue];
-            else opts[firstValue].push(secondValue);
-            // Store a hash of the original key value pairs for later
+            const motorValue = value.substring(value.indexOf('.') + 1);
+            const typeValue = motorValue.indexOf('large') == 0 ? 'large' : 'medium';
+            const portValue = motorValue.indexOf('large') == 0 ? motorValue.substring(5) : motorValue.substring(6);
+            const isDual = portValue.length > 1;
+
+            const text = `${typeValue} motor${isDual ? 's' : ''}|${portValue}`;
+            const key = `${typeValue} motor${isDual ? 's' : ''}`;
+            if (!opts[key]) opts[key] = [];
+            opts[key].push(portValue);
+
             conts[text] = options[opt][0];
             vals[text] = value;
         }
 
-        const currentFirst = this.getFirstValue(this.text_);
-        const currentSecond = this.getSecondValue(this.text_);
+        const currentFirst = this.getFirstValue(this.value_);
+        const currentSecond = this.getSecondValue(this.value_);
 
         if (!this.isFirst_) {
             options = opts[currentFirst];
@@ -432,9 +442,7 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
             options = Object.keys(opts);
             // Flip the first and second options to make it sorted the way we want it (medium, large, dual)
             if (options.length == 3) {
-                const temp = options[1];
-                options[1] = options[0];
-                options[0] = temp;
+                options = [lf("medium motor"), lf("large motor"), lf("large motors")];
             } else {
                 options.reverse();
             }
@@ -448,7 +456,7 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
         const columns = options.length;
 
         for (let i = 0, option: any; option = options[i]; i++) {
-            let text = this.isFirst_ ? option + ' ' + opts[option][0] : currentFirst + ' ' + option;
+            let text = this.isFirst_ ? option + '|' + (option.indexOf('motors') != -1 ? 'BC' : 'A') : currentFirst + '|' + option;
             text = text.replace(/\xA0/g, ' ');
             const content: any = conts[text];
             const value = vals[text];
@@ -466,7 +474,7 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
             button.setAttribute('id', ':' + i); // For aria-activedescendant
             button.setAttribute('role', 'menuitem');
             button.setAttribute('class', 'blocklyDropDownButton');
-            button.title = this.isFirst_ ? this.getFirstValue(content.alt) : content.alt;
+            button.title = this.isFirst_ ? this.getFirstValueI11n(value) : this.getSecondValue(value);
             button.style.width = ((this.itemWidth_) - 8) + 'px';
             button.style.height = ((this.itemWidth_) - 8) + 'px';
             let backgroundColor = this.backgroundColour_;
@@ -495,7 +503,15 @@ export class FieldMotors extends Blockly.FieldDropdown implements Blockly.FieldC
                 contentDiv.removeAttribute('aria-activedescendant');
             });
             let buttonImg = document.createElement('img');
-            buttonImg.src = this.isFirst_ ? isFirstUrl[option.replace(/\xA0/g, ' ')] : content.src;
+            let imgSrc = content.src;
+            if (this.isFirst_) {
+                const motorValue = value.substring(value.indexOf('.') + 1);
+                // Find out what kind of motor this is based on it's value, possible values: mediumX, largeX, and largeXY
+                if (motorValue.indexOf('medium') == 0) imgSrc = isFirstUrl['medium motor'];
+                else if (motorValue.length == 6) imgSrc = isFirstUrl['large motor'];
+                else imgSrc = isFirstUrl['large motors'];
+            }
+            buttonImg.src = imgSrc;
             //buttonImg.alt = icon.alt;
             // Upon click/touch, we will be able to get the clicked element as e.target
             // Store a data attribute on all possible click targets so we can match it to the icon.

libs/core/output.ts

@@ -368,7 +368,8 @@ namespace motors {
         }
 
         private __move(steps: boolean, stepsOrTime: number, speed: number) {
-            step(this._port, {
+            control.dmesg("motor.__move")
+            const p = {
                 useSteps: steps,
                 step1: 0,
                 step2: stepsOrTime,
@@ -376,7 +377,10 @@ namespace motors {
                 speed: this._regulated ? speed : undefined,
                 power: this._regulated ? undefined : speed,
                 useBrake: this._brake
-            })
+            };
+            control.dmesg("motor.1")
+            step(this._port, p)
+            control.dmesg("motor.__move end")
         }
 
         /**
@@ -706,24 +710,36 @@ namespace motors {
     }
 
     function step(out: Output, opts: StepOptions) {
+        control.dmesg('step')
         let op = opts.useSteps ? DAL.opOutputStepSpeed : DAL.opOutputTimeSpeed
         let speed = opts.speed
-        if (speed == null) {
+        if (undefined == speed) {
             speed = opts.power
             op = opts.useSteps ? DAL.opOutputStepPower : DAL.opOutputTimePower
-            if (speed == null)
+            if (undefined == speed)
                 return
         }
         speed = Math.clamp(-100, 100, speed)
+        control.dmesg('speed: ' + speed)
 
         let b = mkCmd(out, op, 15)
+        control.dmesg('STEP 5')
         b.setNumber(NumberFormat.Int8LE, 2, speed)
         // note that b[3] is padding
+        control.dmesg('STEP 1')
         b.setNumber(NumberFormat.Int32LE, 4 + 4 * 0, opts.step1)
+        control.dmesg('STEP 2')
         b.setNumber(NumberFormat.Int32LE, 4 + 4 * 1, opts.step2)
+        control.dmesg('STEP 3')
         b.setNumber(NumberFormat.Int32LE, 4 + 4 * 2, opts.step3)
-        b.setNumber(NumberFormat.Int8LE, 4 + 4 * 3, opts.useBrake ? 1 : 0)
+        control.dmesg('STEP 4')
+        control.dmesg('br ' + opts.useBrake);
+        const br = !!opts.useBrake ? 1 : 0;
+        control.dmesg('Step 4.5 ' + br)
+        b.setNumber(NumberFormat.Int8LE, 4 + 4 * 3, br)
+        control.dmesg('STEP 5')
         writePWM(b)
+        control.dmesg('end step')
     }
 
     const types = [0, 0, 0, 0]

libs/ultrasonic-sensor/docs/reference/sensors/ultrasonic.md

@@ -9,5 +9,5 @@ sensors.ultrasonic1.pauseUntil(UltrasonicSensorEvent.ObjectDetected);
 ## See Also
 
 [on event](/reference/sensors/ultrasonic/on-event),
-[distance](reference/sensors/ultrasonic/distance),
-[pause until](reference/sensors/ultrasonic/pause-until)
+[distance](/reference/sensors/ultrasonic/distance),
+[pause until](/reference/sensors/ultrasonic/pause-until)

package-lock.json

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

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pxt-ev3",
-  "version": "1.0.2",
+  "version": "1.0.8",
   "description": "LEGO MINDSTORMS EV3 for Microsoft MakeCode",
   "private": true,
   "keywords": [
@@ -39,8 +39,8 @@
     "webfonts-generator": "^0.4.0"
   },
   "dependencies": {
-    "pxt-common-packages": "0.23.55",
-    "pxt-core": "4.0.2"
+    "pxt-common-packages": "0.23.56",
+    "pxt-core": "4.0.9"
   },
   "scripts": {
     "test": "node node_modules/pxt-core/built/pxt.js travis"

pxtarget.json

@@ -137,7 +137,6 @@
         "invertedMonaco": false,
         "monacoToolbox": true,
         "invertedToolbox": false,
-        "lightToc": true,
         "coloredToolbox": true,
         "hasAudio": true,
         "saveInMenu": true,