Restrict click event from propagating to parent

GoJS
21

Hi Walter,

With the above hack, I am able to stop the call of my parent click. But at the same time, my trigger is not going into showContextMenu method as well.

We are actually overriding the showContextMenu() method of goJS and using our own set of logic inside that method. Which is not getting executed with this hack.

Without this hack, the trigger goes into showContextMenu() method without any issues.

22

But your click handler is calling your overridden method, isn’t it?

23

On page itself, we are registering this showContextMenu() method by overriding the goJS show() method.

export const registerShowMethodOnContextMenu = (
	goJsContextMenu,
) => {
	if (goJsContextMenu) {
	     goJsContextMenu.show = (node, diagram) => {
			goDiagram = diagram;
			targetNode = node;
			showContextMenu(node, diagram);
		};
	}
};

So on click, the trigger enters this register method and from there on to overridden method showContextMenu().

But with this hack, the trigger is not hitting the goJS show() itself.

24

That is an HTMLInfo, isn’t it?

All of this code isn’t in a subclass of CommandHandler, is it? So you have not overridden the CommandHandler.showContextMenu method, have you? What is your showContextMenu?

25

Yes Walter. goJsContextMenu is HTMLInfo. Anyhow I just returned false for the override method as below,

'contextMenuTool.standardMouseClick': function() {
            return false;
        },

This resolved my issue. Thank you.

26

OK, to be clear, that means you do not need the override I suggested above, yes?

27

Yes Walter. I do not need the suggested workaround anymore. The issue is fixed with the return false.