Skip to content
This repository has been archived by the owner on Sep 1, 2020. It is now read-only.

export show/hideMenu methods in order to customize the trigger #133

Merged
merged 3 commits into from
Jul 20, 2017

Conversation

dCode91
Copy link
Contributor

@dCode91 dCode91 commented Jul 19, 2017

I encounter a problem on triggering the context menu from an element that is not a view itself so it cannot be the child of ContextMenuTrigger. In the JointJS framework, the cell elements are part of paper view, so in order to trigger the contextmenu on the cell elements i need to use the hideMenu and showMenu methods in the cell contexmenu event. Exposing these methods will make the triggering more customized and easier to adapt.

@vkbansal
Copy link
Owner

Can you fix the lint issues?

@dCode91
Copy link
Contributor Author

dCode91 commented Jul 20, 2017

Are you talking about the lint issues in the entire project?(semicolons ";" an "if" statements)

@vkbansal
Copy link
Owner

Nope just the file you changed. It is missing an empty last line.

@dCode91
Copy link
Contributor Author

dCode91 commented Jul 20, 2017

ok, i'll fix that

@vkbansal
Copy link
Owner

/home/travis/build/vkbansal/react-contextmenu/src/index.js
  6:30  error  Multiple spaces found before '}'  no-multi-spaces

@dCode91
Copy link
Contributor Author

dCode91 commented Jul 20, 2017

Done, it looks like everything is ok now

@vkbansal vkbansal merged commit 6248bbd into vkbansal:master Jul 20, 2017
@vkbansal vkbansal mentioned this pull request Mar 29, 2019
@shotforsky
Copy link

shotforsky commented May 9, 2020

Could anyone document this please? really appreciate it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants