Merge branch 'master' into develop
This commit is contained in:
commit
d6ab00783a
|
@ -13,6 +13,7 @@ ___Note:__ Yet to be released changes appear here._
|
|||
* `FEAT`: add horizontal and vertical resize handles
|
||||
* `FEAT`: improve connection cropping (bump to `path-intersection@2`)
|
||||
* `FIX`: correctly mark elements as changed on `{shape|connection}.create` undo
|
||||
* `FIX`: do not open replace menu after multi create ([#1255](https://github.com/bpmn-io/bpmn-js/pull/1255))
|
||||
* `CHORE`: update to `diagram-js@6.2.0`
|
||||
|
||||
## 6.0.7
|
||||
|
|
|
@ -56,9 +56,10 @@ export default function ContextPadProvider(
|
|||
}
|
||||
|
||||
eventBus.on('create.end', 250, function(event) {
|
||||
var shape = event.context.shape;
|
||||
var context = event.context,
|
||||
shape = context.shape;
|
||||
|
||||
if (!hasPrimaryModifier(event)) {
|
||||
if (!hasPrimaryModifier(event) || !contextPad.isOpen(shape)) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -423,6 +424,9 @@ ContextPadProvider.prototype.getContextPadEntries = function(element) {
|
|||
return actions;
|
||||
};
|
||||
|
||||
|
||||
// helpers /////////
|
||||
|
||||
function isEventType(eventBo, type, definition) {
|
||||
|
||||
var isType = eventBo.$instanceOf(type);
|
||||
|
|
|
@ -498,6 +498,31 @@ describe('features - context-pad', function() {
|
|||
}
|
||||
));
|
||||
|
||||
|
||||
it('should NOT open replace menu if context pad NOT open', inject(
|
||||
function(canvas, create, dragging, elementFactory) {
|
||||
|
||||
// given
|
||||
var rootShape = canvas.getRootElement(),
|
||||
startEvent = elementFactory.createShape({ type: 'bpmn:StartEvent' }),
|
||||
task = elementFactory.createShape({ type: 'bpmn:Task' });
|
||||
|
||||
// when
|
||||
create.start(canvasEvent({ x: 0, y: 0 }), [ startEvent, task ]);
|
||||
|
||||
dragging.move(canvasEvent({ x: 50, y: 50 }));
|
||||
dragging.hover({ element: rootShape });
|
||||
dragging.move(canvasEvent({ x: 75, y: 75 }));
|
||||
|
||||
dragging.end(canvasEvent({ x: 75, y: 75 }, { ctrlKey: true, metaKey: true }));
|
||||
|
||||
// then
|
||||
var replaceMenu = domQuery('.bpmn-replace', container);
|
||||
|
||||
expect(replaceMenu).not.to.exist;
|
||||
}
|
||||
));
|
||||
|
||||
});
|
||||
|
||||
});
|
||||
|
|
Loading…
Reference in New Issue