bpmn-js 9.0.1 → 9.0.4

package/CHANGELOG.md

@@ -7,6 +7,24 @@ All notable changes to [bpmn-js](https://github.com/bpmn-io/bpmn-js) are documen
 
 ___Note:__ Yet to be released changes appear here._
 
+## 9.0.4
+
+* `FIX`: remove `label` property on empty label ([#1637](https://github.com/bpmn-io/bpmn-js/issues/1637))
+* `FIX`: create drilldown overlays on `viewer.open` ([`574a67438`](https://github.com/bpmn-io/bpmn-js/commit/574a674381d6449b509396b6d17c4ca94674ea1c))
+* `FIX`: render data association inside collapsed sub-processes ([#1619](https://github.com/bpmn-io/bpmn-js/issues/1619))
+* `FIX`: preserve multi-instance properties when toggling between parallel and sequential ([#1581](https://github.com/bpmn-io/bpmn-js/issues/1581))
+* `FIX`: correct hanging sequence flow label after collapsing sub-process ([#1617](https://github.com/bpmn-io/bpmn-js/issues/1617))
+* `FIX`: correct start event not added to newly created sub-process ([#1631](https://github.com/bpmn-io/bpmn-js/issues/1631))
+
+## 9.0.3
+
+* `FIX`: submit direct editing result on drilldown ([#1609](https://github.com/bpmn-io/bpmn-js/issues/1609))
+* `DEPS`: bump to `diagram-js@8.2.0` ([2bac149](https://github.com/bpmn-io/bpmn-js/commit/2bac1495058601fec203c134b41efe5600e5fc97))
+
+## 9.0.2
+
+* `FIX`: support modeling of groups in collapsed subporcesses ([#1606](https://github.com/bpmn-io/bpmn-js/issues/1606))
+* `FIX`: override default padding of breadcrumb element ([#1608](https://github.com/bpmn-io/bpmn-js/pull/1608))
 
 ## 9.0.1
 
@@ -14,20 +32,19 @@ ___Note:__ Yet to be released changes appear here._
 * `FIX`: support DIs without associated business object ([#1605](https://github.com/bpmn-io/bpmn-js/pull/1605))
 * `DEPS`: bump to `diagram-js@8.1.2` ([bdf9cf3](https://github.com/bpmn-io/bpmn-js/commit/bdf9cf3e752254a4c8172031d8a493955a9fca9c))
 
-
 ## 9.0.0
 
 * `FEAT`: support drilldown and modeling of collapsed subprocesses ([#1443](https://github.com/bpmn-io/bpmn-js/issues/1443))
 * `FEAT`: update embedded label bounds when shape is moved ([#1586](https://github.com/bpmn-io/bpmn-js/pull/1586))
 * `FIX`: create di for embedded labels ([#1579](https://github.com/bpmn-io/bpmn-js/pull/1579))
 * `CHORE`: expose `BpmnRenderer` extension points ([#1585](https://github.com/bpmn-io/bpmn-js/pull/1585))
-* `CHORE`: bump to `diagram-js@8.1.1`
+* `DEPS`: bump to `diagram-js@8.1.1`
 
 ### Breaking Changes
 
-* Reworked the link of elements to bpmn DIs. You must access the `di` directly from the diagram element instead of the `businessObject`. [#1472](https://github.com/bpmn-io/bpmn-js/issues/1472)
-* Reworked `viewer.open` behavior for single planes. ([#1576](https://github.com/bpmn-io/bpmn-js/pull/1576))
-* Reworked import and `BpmnFactory` APIs. [#1472](https://github.com/bpmn-io/bpmn-js/issues/1472)
+* Reworked the link of elements to bpmn DIs. You must access the `di` directly from the diagram element instead of the `businessObject` [#1472](https://github.com/bpmn-io/bpmn-js/issues/1472).
+* Reworked `viewer.open` behavior for single planes ([#1576](https://github.com/bpmn-io/bpmn-js/pull/1576)).
+* Reworked import and `BpmnFactory` APIs [#1472](https://github.com/bpmn-io/bpmn-js/issues/1472).
 * Added `bpmn-js.css`, which is required to display drilldown overlays correctly.
 
 ## 8.10.0

package/dist/assets/bpmn-js.css

@@ -40,7 +40,10 @@
   display: none;
   flex-wrap: wrap;
   align-items: center;
-  top: 10px;
+  top: 30px;
+  left: 30px;
+  padding: 0px;
+  margin: 0px;
   font-family: var(--breadcrumbs-font-family);
   font-size: 16px;
   line-height: normal;
@@ -51,11 +54,11 @@
 }
 
 .djs-palette-shown .bjs-breadcrumbs {
-  left: 50px;
+  left: 90px;
 }
 
 .djs-palette-shown.djs-palette-two-column .bjs-breadcrumbs {
-  left: 100px;
+  left: 140px;
 }
 
 .bjs-breadcrumbs li {

package/dist/bpmn-modeler.development.js

@@ -1,5 +1,5 @@
 /*!
- * bpmn-js - bpmn-modeler v9.0.1
+ * bpmn-js - bpmn-modeler v9.0.4
  *
  * Copyright (c) 2014-present, camunda Services GmbH
  *
@@ -8,7 +8,7 @@
  *
  * Source Code: https://github.com/bpmn-io/bpmn-js
  *
- * Date: 2022-02-18
+ * Date: 2022-05-13
  */
 (function (global, factory) {
 	typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
@@ -2372,7 +2372,7 @@
 	    }
 	  });
 
-	  eventBus.on([ 'render.getShapePath', 'render.getConnectionPath'], renderPriority, function(evt, element) {
+	  eventBus.on([ 'render.getShapePath', 'render.getConnectionPath' ], renderPriority, function(evt, element) {
 	    if (self.canRender(element)) {
 	      if (evt.type === 'render.getShapePath') {
 	        return self.getShapePath(element);
@@ -2684,7 +2684,7 @@
 
 	  stopRecursion = !!stopRecursion;
 	  if (!isArray$2(elements)) {
-	    elements = [elements];
+	    elements = [ elements ];
 	  }
 
 	  var minX,
@@ -2864,11 +2864,11 @@
 	      height = shape.height;
 
 	  var shapePath = [
-	    ['M', x, y],
-	    ['l', width, 0],
-	    ['l', 0, height],
-	    ['l', -width, 0],
-	    ['z']
+	    [ 'M', x, y ],
+	    [ 'l', width, 0 ],
+	    [ 'l', 0, height ],
+	    [ 'l', -width, 0 ],
+	    [ 'z' ]
 	  ];
 
 	  return componentsToPath(shapePath);
@@ -4594,6 +4594,25 @@
 	  return layer.group;
 	};
 
+	/**
+	 * For a given index, return the number of layers that have a higher index and
+	 * are visible.
+	 *
+	 * This is used to determine the node a layer should be inserted at.
+	 *
+	 * @param {Number} index
+	 * @returns {Number}
+	 */
+	Canvas.prototype._getChildIndex = function(index) {
+	  return reduce(this._layers, function(childIndex, layer) {
+	    if (layer.visible && index >= layer.index) {
+	      childIndex++;
+	    }
+
+	    return childIndex;
+	  }, 0);
+	};
+
 	/**
 	 * Creates a given layer and returns it.
 	 *
@@ -4608,19 +4627,80 @@
 	    index = UTILITY_LAYER_INDEX;
 	  }
 
-	  var childIndex = reduce(this._layers, function(childIndex, layer) {
-	    if (index >= layer.index) {
-	      childIndex++;
-	    }
-
-	    return childIndex;
-	  }, 0);
+	  var childIndex = this._getChildIndex(index);
 
 	  return {
 	    group: createGroup(this._viewport, 'layer-' + name, childIndex),
-	    index: index
+	    index: index,
+	    visible: true
 	  };
+	};
+
 
+	/**
+	 * Shows a given layer.
+	 *
+	 * @param {String} layer
+	 * @returns {SVGElement}
+	 */
+	Canvas.prototype.showLayer = function(name) {
+
+	  if (!name) {
+	    throw new Error('must specify a name');
+	  }
+
+	  var layer = this._layers[name];
+
+	  if (!layer) {
+	    throw new Error('layer <' + name + '> does not exist');
+	  }
+
+	  var viewport = this._viewport;
+	  var group = layer.group;
+	  var index = layer.index;
+
+	  if (layer.visible) {
+	    return group;
+	  }
+
+	  var childIndex = this._getChildIndex(index);
+
+	  viewport.insertBefore(group, viewport.childNodes[childIndex] || null);
+
+	  layer.visible = true;
+
+	  return group;
+	};
+
+	/**
+	 * Hides a given layer.
+	 *
+	 * @param {String} layer
+	 * @returns {SVGElement}
+	 */
+	Canvas.prototype.hideLayer = function(name) {
+
+	  if (!name) {
+	    throw new Error('must specify a name');
+	  }
+
+	  var layer = this._layers[name];
+
+	  if (!layer) {
+	    throw new Error('layer <' + name + '> does not exist');
+	  }
+
+	  var group = layer.group;
+
+	  if (!layer.visible) {
+	    return group;
+	  }
+
+	  remove$1(group);
+
+	  layer.visible = false;
+
+	  return group;
 	};
 
 
@@ -4863,7 +4943,7 @@
 
 	  var layer = this.getLayer(layerName, PLANE_LAYER_INDEX);
 
-	  attr(layer, 'display', 'none');
+	  this.hideLayer(layerName);
 
 	  this._addRoot(rootElement, layer);
 
@@ -4982,17 +5062,13 @@
 
 	  var currentRoot = this._rootElement;
 
-	  var currentLayer;
-
 	  if (currentRoot) {
 
 	    // un-associate previous root element <svg>
 	    this._elementRegistry.updateGraphics(currentRoot, null, true);
 
 	    // hide previous layer
-	    currentLayer = this._findPlaneForRoot(currentRoot).layer;
-
-	    attr(currentLayer, 'display', 'none');
+	    this.hideLayer(currentRoot.layer);
 	  }
 
 	  if (rootElement) {
@@ -5005,7 +5081,7 @@
 	    this._elementRegistry.updateGraphics(rootElement, this._svg, true);
 
 	    // show root layer
-	    attr(layer, 'display', null);
+	    this.showLayer(rootElement.layer);
 	  }
 
 	  this._rootElement = rootElement;
@@ -7403,7 +7479,7 @@
 	  options = options || {};
 
 	  var configModule = {
-	    'config': ['value', options]
+	    'config': [ 'value', options ]
 	  };
 
 	  var modules = [ configModule, CoreModule$1 ].concat(options.modules || []);
@@ -15707,7 +15783,7 @@
 	}
 
 	/**
-	 * Returns all diagrams in the same hirarchy as the requested diagram.
+	 * Returns all diagrams in the same hierarchy as the requested diagram.
 	 * Includes all parent and sub process diagrams.
 	 *
 	 * @param {Array} definitions
@@ -20962,7 +21038,7 @@
 	};
 
 
-	Outline.$inject = ['eventBus', 'styles', 'elementRegistry'];
+	Outline.$inject = [ 'eventBus', 'styles', 'elementRegistry' ];
 
 	var OutlineModule = {
 	  __init__: [ 'outline' ],
@@ -21992,7 +22068,7 @@
 	];
 
 	var ChangeSupportModule = {
-	  __init__: [ 'changeSupport'],
+	  __init__: [ 'changeSupport' ],
 	  changeSupport: [ 'type', ChangeSupport ]
 	};
 
@@ -22164,7 +22240,7 @@
 
 	inherits$1(RootElementsBehavior, CommandInterceptor);
 
-	RootElementsBehavior.$inject = [ 'canvas', 'injector'];
+	RootElementsBehavior.$inject = [ 'canvas', 'injector' ];
 
 	var RootElementsModule = {
 	  __init__: [ 'rootElementsBehavior' ],
@@ -22667,7 +22743,7 @@
 	      collapsedElements.push(bo);
 	    }
 
-	    if (is$1(parent, 'bpmn:SubProcess') && parent !== plane.bpmnElement) {
+	    if (shouldMoveToPlane(bo, plane)) {
 
 	      // don't change the array while we iterate over it
 	      elementsToMove.push({ diElement: diElement, parent: parent });
@@ -22756,7 +22832,7 @@
 	SubprocessCompatibility.$inject = [ 'eventBus', 'moddle' ];
 
 
-	// helpers
+	// helpers //////////////////////////
 
 	function findRootDiagram(element) {
 	  if (is$1(element, 'bpmndi:BPMNDiagram')) {
@@ -22788,6 +22864,23 @@
 	  return asBounds(planeTrbl);
 	}
 
+	function shouldMoveToPlane(bo, plane) {
+	  var parent = bo.$parent;
+
+	  // don't move elements that are already on the plane
+	  if (!is$1(parent, 'bpmn:SubProcess') || parent === plane.bpmnElement) {
+	    return false;
+	  }
+
+	  // dataAssociations are children of the subprocess but rendered on process level
+	  // cf. https://github.com/bpmn-io/bpmn-js/issues/1619
+	  if (isAny(bo, ['bpmn:DataInputAssociation', 'bpmn:DataOutputAssociation'])) {
+	    return false;
+	  }
+
+	  return true;
+	}
+
 	var LOW_PRIORITY$l = 250;
 	var ARROW_DOWN_SVG = '<svg xmlns="http://www.w3.org/2000/svg" width="20" height="20" viewBox="0 0 16 16"><path fill-rule="evenodd" d="M4.81801948,3.50735931 L10.4996894,9.1896894 L10.5,4 L12,4 L12,12 L4,12 L4,10.5 L9.6896894,10.4996894 L3.75735931,4.56801948 C3.46446609,4.27512627 3.46446609,3.80025253 3.75735931,3.50735931 C4.05025253,3.21446609 4.52512627,3.21446609 4.81801948,3.50735931 Z"/></svg>';
 
@@ -22863,7 +22956,7 @@
 	    }, true);
 
 
-	  eventBus.on('import.done', function() {
+	  eventBus.on('import.render.complete', function() {
 	    elementRegistry.filter(function(e) {
 	      return self.canDrillDown(e);
 	    }).map(function(el) {
@@ -23273,7 +23366,7 @@
 	var KEYCODE_Y = 89;
 	var KEYCODE_Z = 90;
 
-	var KEYS_COPY = ['c', 'C', KEYCODE_C ];
+	var KEYS_COPY = [ 'c', 'C', KEYCODE_C ];
 	var KEYS_PASTE = [ 'v', 'V', KEYCODE_V ];
 	var KEYS_REDO = [ 'y', 'Y', KEYCODE_Y ];
 	var KEYS_UNDO = [ 'z', 'Z', KEYCODE_Z ];
@@ -23429,7 +23522,7 @@
 
 	    var event = context.keyEvent;
 
-	    if (isKey(['Backspace', 'Delete', 'Del' ], event)) {
+	    if (isKey([ 'Backspace', 'Delete', 'Del' ], event)) {
 	      editorActions.trigger('removeSelection');
 
 	      return true;
@@ -29179,11 +29272,11 @@
 	      y = center.y;
 
 	  return [
-	    ['M', x, y],
-	    ['m', 0, -r],
-	    ['a', r, r, 0, 1, 1, 0, 2 * r],
-	    ['a', r, r, 0, 1, 1, 0, -2 * r],
-	    ['z']
+	    [ 'M', x, y ],
+	    [ 'm', 0, -r ],
+	    [ 'a', r, r, 0, 1, 1, 0, 2 * r ],
+	    [ 'a', r, r, 0, 1, 1, 0, -2 * r ],
+	    [ 'z' ]
 	  ];
 	}
 
@@ -30759,7 +30852,7 @@
 	  return parent.children || [];
 	}
 
-	var abs$5= Math.abs,
+	var abs$5 = Math.abs,
 	    round$6 = Math.round;
 
 	var TOLERANCE = 10;
@@ -36587,16 +36680,16 @@
 	  var translate = this._translate;
 
 	  function toggleLoopEntry(event, entry) {
-	    var loopCharacteristics;
+	    var loopCharacteristics = getBusinessObject(element).loopCharacteristics;
 
 	    if (entry.active) {
 	      loopCharacteristics = undefined;
 	    } else {
-	      loopCharacteristics = self._moddle.create(entry.options.loopCharacteristics);
-
-	      if (entry.options.isSequential) {
-	        loopCharacteristics.isSequential = entry.options.isSequential;
+	      if (isUndefined$1(entry.options.isSequential) || !loopCharacteristics) {
+	        loopCharacteristics = self._moddle.create(entry.options.loopCharacteristics);
 	      }
+
+	      loopCharacteristics.isSequential = entry.options.isSequential;
 	    }
 	    self._modeling.updateProperties(element, { loopCharacteristics: loopCharacteristics });
 	  }
@@ -40119,7 +40212,8 @@
 	    'canvas.viewbox.changing',
 	    'drag.init',
 	    'element.mousedown',
-	    'popupMenu.open'
+	    'popupMenu.open',
+	    'root.set'
 	  ], function(event) {
 
 	    if (directEditing.isActive()) {
@@ -42337,7 +42431,7 @@
 	 */
 	function GroupBehavior(
 	    bpmnFactory,
-	    canvas,
+	    bpmnjs,
 	    elementRegistry,
 	    eventBus,
 	    injector,
@@ -42345,18 +42439,6 @@
 	) {
 	  injector.invoke(CommandInterceptor, this);
 
-	  /**
-	   * Gets process definitions
-	   *
-	   * @return {ModdleElement} definitions
-	   */
-	  function getDefinitions() {
-	    var rootElement = canvas.getRootElement(),
-	        businessObject = getBusinessObject(rootElement);
-
-	    return businessObject.$parent;
-	  }
-
 	  /**
 	   * Removes a referenced category value for a given group shape
 	   *
@@ -42392,7 +42474,7 @@
 	   */
 	  function removeCategory(category) {
 
-	    var definitions = getDefinitions();
+	    var definitions = bpmnjs.getDefinitions();
 
 	    remove(definitions.get('rootElements'), category);
 	  }
@@ -42457,7 +42539,7 @@
 
 	      var businessObject = getBusinessObject(shape),
 	          categoryValueRef = businessObject.categoryValueRef,
-	          definitions = getDefinitions(),
+	          definitions = bpmnjs.getDefinitions(),
 	          category = categoryValueRef ? categoryValueRef.$parent : null;
 
 	      add(category.get('categoryValue'), categoryValueRef);
@@ -42475,7 +42557,7 @@
 
 	    if (is$1(businessObject, 'bpmn:Group') && !businessObject.categoryValueRef) {
 
-	      var definitions = getDefinitions(),
+	      var definitions = bpmnjs.getDefinitions(),
 	          categoryValue = createCategoryValue(definitions, bpmnFactory);
 
 	      // link the reference to the Group
@@ -42503,7 +42585,7 @@
 	        categoryValue;
 
 	    if (is$1(property, 'bpmn:CategoryValue')) {
-	      categoryValue = createCategoryValue(getDefinitions(), bpmnFactory);
+	      categoryValue = createCategoryValue(bpmnjs.getDefinitions(), bpmnFactory);
 
 	      // return copy of category
 	      return moddleCopy.copyElement(property, categoryValue);
@@ -42514,7 +42596,7 @@
 
 	GroupBehavior.$inject = [
 	  'bpmnFactory',
-	  'canvas',
+	  'bpmnjs',
 	  'elementRegistry',
 	  'eventBus',
 	  'injector',
@@ -45400,7 +45482,7 @@
 
 	    if (
 	      !is$1(newShape, 'bpmn:SubProcess') ||
-	      !is$1(oldShape, 'bpmn:Task') ||
+	      ! (is$1(oldShape, 'bpmn:Task') || is$1(oldShape, 'bpmn:CallActivity')) ||
 	      !isExpanded(newShape)
 	    ) {
 	      return;
@@ -47137,7 +47219,7 @@
 	    // render labels always on top
 	    if (element.labelTarget) {
 	      return {
-	        parent: canvas.findRoot(element.labelTarget) || canvas.getRootElement(),
+	        parent: canvas.findRoot(newParent) || canvas.getRootElement(),
 	        index: -1
 	      };
 	    }
@@ -48258,7 +48340,7 @@
 	}
 
 	var LabelSupportModule = {
-	  __init__: [ 'labelSupport'],
+	  __init__: [ 'labelSupport' ],
 	  labelSupport: [ 'type', LabelSupport ]
 	};
 
@@ -48446,7 +48528,7 @@
 	    saveClear(oldShape.attachers, function(attacher) {
 	      var allowed = rules.allowed('elements.move', {
 	        target: newShape,
-	        shapes: [attacher]
+	        shapes: [ attacher ]
 	      });
 
 	      if (allowed === 'attach') {
@@ -49580,7 +49662,7 @@
 	}
 
 	var SpaceToolModule = {
-	  __init__: ['spaceToolPreview'],
+	  __init__: [ 'spaceToolPreview' ],
 	  __depends__: [
 	    DraggingModule,
 	    RulesModule$1,
@@ -49588,8 +49670,8 @@
 	    PreviewSupportModule,
 	    MouseModule
 	  ],
-	  spaceTool: ['type', SpaceTool ],
-	  spaceToolPreview: ['type', SpaceToolPreview ]
+	  spaceTool: [ 'type', SpaceTool ],
+	  spaceToolPreview: [ 'type', SpaceToolPreview ]
 	};
 
 	function BpmnFactory(moddle) {
@@ -51709,7 +51791,7 @@
 	      newParentIndex = context.newParentIndex,
 	      oldParent = shape.parent;
 
-	  context.oldBounds = pick(shape, [ 'x', 'y', 'width', 'height']);
+	  context.oldBounds = pick(shape, [ 'x', 'y', 'width', 'height' ]);
 
 	  // save old parent in context
 	  context.oldParent = oldParent;
@@ -52355,7 +52437,7 @@
 	  // recursively hide/show children
 	  var result = setHiddenRecursive(children, shape.collapsed);
 
-	  return [shape].concat(result);
+	  return [ shape ].concat(result);
 	};
 
 
@@ -52372,7 +52454,7 @@
 	  // retoggle state
 	  shape.collapsed = !shape.collapsed;
 
-	  return [shape].concat(result);
+	  return [ shape ].concat(result);
 	};
 
 
@@ -54054,7 +54136,7 @@
 	    }
 
 	    if (!text && di.label) {
-	      di.label = null;
+	      delete di.label;
 	    }
 	  }
 
@@ -58973,8 +59055,8 @@
 	  var h = box.height + offset * 2;
 
 	  var styles = [
-	    'width: '+ w +'px',
-	    'height: '+ h + 'px'
+	    'width: ' + w + 'px',
+	    'height: ' + h + 'px'
 	  ].join('; ');
 
 	  return {