pxt-core 7.3.21 → 7.3.22

package/built/pxtblocks.d.ts

@@ -471,7 +471,7 @@ declare namespace pxtblockly {
         protected animateRef: any;
         protected asset: pxt.Animation;
         protected initInterval: number;
-        initView(): void;
+        init(): void;
         showEditor_(): void;
         render_(): void;
         protected getAssetType(): pxt.AssetType;
@@ -508,7 +508,7 @@ declare namespace pxtblockly {
         CURSOR: string;
         private type_;
         constructor(state: string, params: Blockly.FieldCustomOptions, opt_validator?: Function);
-        initView(): void;
+        init(): void;
         updateSize_(): void;
         /**
          * Return 'TRUE' if the toggle is ON, 'FALSE' otherwise.
@@ -1243,7 +1243,7 @@ declare namespace pxtblockly {
         protected blocksInfo: pxtc.BlocksInfo;
         protected transparent: TilesetDropdownOption;
         constructor(text: string, options: FieldImageDropdownOptions, validator?: Function);
-        initView(): void;
+        init(): void;
         getValue(): any;
         getText(): string;
         render_(): void;
@@ -1265,7 +1265,7 @@ declare namespace pxtblockly {
         CURSOR: string;
         private type_;
         constructor(state: string, params: Blockly.FieldCustomOptions, opt_validator?: Function);
-        initView(): void;
+        init(): void;
         getDisplayText_(): string;
         getTrueText(): string;
         getFalseText(): string;

package/built/pxtblocks.js

@@ -1497,7 +1497,7 @@ var pxt;
         function compileBlockAsync(b, blockInfo) {
             const w = b.workspace;
             const e = mkEnv(w, blockInfo);
-            infer(w && w.getAllBlocks(false), e, w);
+            infer(w && w.getAllBlocks(), e, w);
             const compiled = compileStatementBlock(e, b);
             removeAllPlaceholders();
             return tdASTtoTS(e, compiled);
@@ -1518,7 +1518,7 @@ var pxt;
         function compileWorkspace(e, w, blockInfo) {
             try {
                 // all compiled top level blocks are events
-                let allBlocks = w.getAllBlocks(false);
+                let allBlocks = w.getAllBlocks();
                 if (pxt.react.getTilemapProject) {
                     pxt.react.getTilemapProject().removeInactiveBlockAssets(allBlocks.map(b => b.id));
                 }
@@ -1788,11 +1788,13 @@ var pxt;
             });
         }
         function maybeAddComment(b, comments) {
-            var _a;
-            // Check if getCommentText exists, block may be placeholder
-            const text = (_a = b.getCommentText) === null || _a === void 0 ? void 0 : _a.call(b);
-            if (text) {
-                comments.push(text);
+            if (b.comment) {
+                if ((typeof b.comment) === "string") {
+                    comments.push(b.comment);
+                }
+                else {
+                    comments.push(b.comment.getText());
+                }
             }
         }
         function addCommentNodes(comments, r) {
@@ -2455,9 +2457,9 @@ var pxt;
                 }
             });
             // we'll ignore disabled blocks in the final output
-            const oldBlocks = oldWs.getAllBlocks(false).filter(b => b.isEnabled());
+            const oldBlocks = oldWs.getAllBlocks().filter(b => b.isEnabled());
             const oldTopBlocks = oldWs.getTopBlocks(false).filter(b => b.isEnabled());
-            const newBlocks = newWs.getAllBlocks(false).filter(b => b.isEnabled());
+            const newBlocks = newWs.getAllBlocks().filter(b => b.isEnabled());
             log(`blocks`, newBlocks.map(b => b.toDevString()));
             log(newBlocks);
             if (oldBlocks.length == 0 && newBlocks.length == 0) {
@@ -2479,11 +2481,11 @@ var pxt;
             const newXml = pxt.blocks.saveWorkspaceXml(newWs, true);
             pxt.blocks.domToWorkspaceNoEvents(Blockly.Xml.textToDom(newXml), ws);
             // delete disabled blocks from final workspace
-            ws.getAllBlocks(false).filter(b => !b.isEnabled()).forEach(b => {
+            ws.getAllBlocks().filter(b => !b.isEnabled()).forEach(b => {
                 log('disabled ', b.toDevString());
                 b.dispose(false);
             });
-            const todoBlocks = pxt.Util.toDictionary(ws.getAllBlocks(false), b => b.id);
+            const todoBlocks = pxt.Util.toDictionary(ws.getAllBlocks(), b => b.id);
             log(`todo blocks`, todoBlocks);
             logTodo('start');
             // 1. deleted top blocks
@@ -2493,7 +2495,7 @@ var pxt;
                     done(b);
                     const b2 = cloneIntoDiff(b);
                     done(b2);
-                    b2.setEnabled(false);
+                    b2.setDisabled(true);
                 });
                 logTodo('deleted top');
             }
@@ -2558,7 +2560,7 @@ var pxt;
             });
             logTodo('unmodified');
             // if nothing is left in the workspace, we "missed" change
-            if (!ws.getAllBlocks(false).length) {
+            if (!ws.getAllBlocks().length) {
                 pxt.tickEvent("blocks.diff", { missed: 1 });
                 return {
                     ws,
@@ -2591,7 +2593,7 @@ var pxt;
             function stitch(b) {
                 log(`stitching ${b.toDevString()}->${dids[b.id]}`);
                 const wb = ws.getBlockById(dids[b.id]);
-                wb.setEnabled(false);
+                wb.setDisabled(true);
                 markUsed(wb);
                 done(wb);
                 // connect previous connection to delted or existing block
@@ -2853,10 +2855,10 @@ var pxt;
         function applyMetaComments(workspace) {
             // process meta comments
             // @highlight -> highlight block
-            workspace.getAllBlocks(false)
-                .filter(b => !!b.getCommentText())
+            workspace.getAllBlocks()
+                .filter(b => !!b.comment && b.comment instanceof Blockly.Comment)
                 .forEach(b => {
-                const c = b.getCommentText();
+                const c = b.comment.getText();
                 if (/@highlight/.test(c)) {
                     const cc = c.replace(/@highlight/g, '').trim();
                     b.setCommentText(cc || null);
@@ -4341,7 +4343,7 @@ var pxt;
                                 }
                                 else if (pr.type == "number" && pr.shadowBlockId && pr.shadowBlockId == "value") {
                                     inputName = undefined;
-                                    fields.push(namedField(new Blockly.FieldNumber("0"), defName));
+                                    fields.push(namedField(new Blockly.FieldTextInput("0", Blockly.FieldTextInput.numberValidator), defName));
                                 }
                                 else if (pr.type == "string" && pr.shadowOptions && pr.shadowOptions.toString) {
                                     inputCheck = null;
@@ -5443,6 +5445,11 @@ var pxt;
                         let blockText = '<xml>' +
                             '<block type="variables_change" gap="' + gap + '">' +
                             Blockly.Variables.generateVariableFieldXmlString(mostRecentVariable) +
+                            '<value name="DELTA">' +
+                            '<shadow type="math_number">' +
+                            '<field name="NUM">1</field>' +
+                            '</shadow>' +
+                            '</value>' +
                             '</block>' +
                             '</xml>';
                         let block = Blockly.Xml.textToDom(blockText).firstChild;
@@ -7615,8 +7622,19 @@ var pxtblockly;
                 this.valueText = text;
         }
         init() {
-            super.init();
+            if (this.isInitialized())
+                return;
+            // Build the DOM.
+            this.fieldGroup_ = Blockly.utils.dom.createSvgElement('g', {}, null);
+            if (!this.visible_) {
+                this.fieldGroup_.style.display = 'none';
+            }
             this.onInit();
+            this.updateEditable();
+            this.sourceBlock_.getSvgRoot().appendChild(this.fieldGroup_);
+            // Force a render.
+            this.render_();
+            this.mouseDownWrapper_ = Blockly.bindEventWithChecks_(this.getClickTarget_(), "mousedown", this, this.onMouseDown_);
         }
         dispose() {
             this.onDispose();
@@ -7769,11 +7787,9 @@ var pxtblockly;
         }
         getDisplayText_() {
             // This is only used when isGreyBlock is true
-            if (this.isGreyBlock) {
-                const text = pxt.Util.htmlUnescape(this.valueText);
-                return text.substr(0, text.indexOf("(")) + "(...)";
-            }
-            return "";
+            const text = pxt.Util.htmlUnescape(this.valueText);
+            return text.substr(0, text.indexOf("(")) + "(...)";
+            ;
         }
         updateEditable() {
             if (this.isGreyBlock && this.fieldGroup_) {
@@ -8030,7 +8046,12 @@ var pxtblockly;
                 }
             };
         }
-        initView() {
+        init() {
+            if (this.fieldGroup_) {
+                // Field has already been initialized once.
+                return;
+            }
+            super.init();
             // Register mouseover events for animating preview
             this.sourceBlock_.getSvgRoot().addEventListener("mouseenter", this.onMouseEnter);
             this.sourceBlock_.getSvgRoot().addEventListener("mouseleave", this.onMouseLeave);
@@ -8236,10 +8257,16 @@ var pxtblockly;
             this.addArgType('toggle');
             this.type_ = params.type;
         }
-        initView() {
-            if (!this.fieldGroup_) {
+        init() {
+            if (this.fieldGroup_) {
+                // Field has already been initialized once.
                 return;
             }
+            // Build the DOM.
+            this.fieldGroup_ = Blockly.utils.dom.createSvgElement('g', {}, null);
+            if (!this.visible_) {
+                this.fieldGroup_.style.display = 'none';
+            }
             // Add an attribute to cassify the type of field.
             if (this.getArgTypes() !== null) {
                 if (this.sourceBlock_.isShadow()) {
@@ -8268,10 +8295,15 @@ var pxtblockly;
                 'dy': '0.6ex',
                 'y': size.height / 2
             }, this.fieldGroup_);
+            this.updateEditable();
+            this.sourceBlock_.getSvgRoot().appendChild(this.fieldGroup_);
             this.switchToggle(this.state_);
             this.setValue(this.getValue());
             // Force a render.
-            this.markDirty();
+            this.render_();
+            this.size_.width = 0;
+            this.mouseDownWrapper_ =
+                Blockly.bindEventWithChecks_(this.getClickTarget_(), 'mousedown', this, this.onMouseDown_);
         }
         updateSize_() {
             this.size_.width = 30;
@@ -9741,7 +9773,7 @@ var pxtblockly;
         }
         render_() {
             if (!this.visible_) {
-                this.markDirty();
+                this.size_.width = 0;
                 return;
             }
             if (!this.elt) {
@@ -11245,7 +11277,7 @@ var pxtblockly;
         dropdownCreate() {
             let functionList = [];
             if (this.sourceBlock_ && this.sourceBlock_.workspace) {
-                let blocks = this.sourceBlock_.workspace.getAllBlocks(false);
+                let blocks = this.sourceBlock_.workspace.getAllBlocks();
                 // Iterate through every block and check the name.
                 for (let i = 0; i < blocks.length; i++) {
                     if (blocks[i].getProcedureDef) {
@@ -11274,6 +11306,10 @@ var pxtblockly;
         }
         onItemSelected(menu, menuItem) {
             let itemText = menuItem.getValue();
+            if (this.sourceBlock_) {
+                // Call any validation function, and allow it to override.
+                itemText = this.callValidator(itemText);
+            }
             if (itemText !== null) {
                 this.setValue(itemText);
             }
@@ -11768,8 +11804,8 @@ var pxtblockly;
             }
             return FieldTileset.referencedTiles;
         }
-        initView() {
-            super.initView();
+        init() {
+            super.init();
             if (this.sourceBlock_ && this.sourceBlock_.isInFlyout) {
                 this.setValue(this.getOptions()[0][1]);
             }
@@ -11918,10 +11954,16 @@ var pxtblockly;
             this.addArgType('toggle');
             this.type_ = params.type;
         }
-        initView() {
-            if (!this.fieldGroup_) {
+        init() {
+            if (this.fieldGroup_) {
+                // Field has already been initialized once.
                 return;
             }
+            // Build the DOM.
+            this.fieldGroup_ = Blockly.utils.dom.createSvgElement('g', {}, null);
+            if (!this.visible_) {
+                this.fieldGroup_.style.display = 'none';
+            }
             // Add an attribute to cassify the type of field.
             if (this.getArgTypes() !== null) {
                 if (this.sourceBlock_.isShadow()) {
@@ -11993,7 +12035,10 @@ var pxtblockly;
             this.switchToggle(this.state_);
             this.setValue(this.getValue());
             // Force a render.
-            this.markDirty();
+            this.render_();
+            this.size_.width = 0;
+            this.mouseDownWrapper_ =
+                Blockly.bindEventWithChecks_(this.getClickTarget_(), 'mousedown', this, this.onMouseDown_);
         }
         getDisplayText_() {
             return this.state_ ? this.getTrueText() : this.getFalseText();