diff --git a/core/flyout.js b/core/flyout.js index 0484b53fc..a216e60f5 100644 --- a/core/flyout.js +++ b/core/flyout.js @@ -637,18 +637,37 @@ Blockly.Flyout.prototype.show = function(xmlList) { // Create the blocks to be shown in this flyout. var blocks = []; var gaps = []; + var lastBlock = null; this.permanentlyDisabled_.length = 0; for (var i = 0, xml; xml = xmlList[i]; i++) { - if (xml.tagName && xml.tagName.toUpperCase() == 'BLOCK') { - var curBlock = Blockly.Xml.domToBlock(xml, this.workspace_); - if (curBlock.disabled) { - // Record blocks that were initially disabled. - // Do not enable these blocks as a result of capacity filtering. - this.permanentlyDisabled_.push(curBlock); + if (xml.tagName) { + if (xml.tagName.toUpperCase() == 'BLOCK') { + lastBlock = xml; + var curBlock = Blockly.Xml.domToBlock(xml, this.workspace_); + if (curBlock.disabled) { + // Record blocks that were initially disabled. + // Do not enable these blocks as a result of capacity filtering. + this.permanentlyDisabled_.push(curBlock); + } + blocks.push(curBlock); + var gap = parseInt(xml.getAttribute('gap'), 10); + gaps.push(isNaN(gap) ? this.MARGIN * 3 : gap); + } else if (xml.tagName.toUpperCase() == 'SEP') { + // Change the gap between two blocks. + // + // The default gap is 24, can be set larger or smaller. + // Note that a deprecated method is to add a gap to a block. + // + var newGap = parseInt(xml.getAttribute('gap'), 10); + // Ignore gaps before the first block. + if (!isNaN(newGap) && lastBlock) { + var oldGap = parseInt(lastBlock.getAttribute('gap')); + var gap = isNaN(oldGap) ? newGap : oldGap + newGap; + gaps[gaps.length - 1] = gap; + } else { + gaps.push(this.MARGIN * 3); + } } - blocks.push(curBlock); - var gap = parseInt(xml.getAttribute('gap'), 10); - gaps.push(isNaN(gap) ? this.MARGIN * 3 : gap); } }