Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

If the Context Menu is already open, and you again right click, then it opens system context menu #78

Open
ankitsalu opened this issue Apr 10, 2019 · 8 comments

Comments

@ankitsalu
Copy link

No description provided.

@msarsha
Copy link
Owner

msarsha commented Apr 11, 2019

True, it's the default behavior and I see no reason to prevent it from opening.
Would love to discuss if you have another opinion 🙂

@ankitsalu
Copy link
Author

ankitsalu commented Apr 11, 2019

Thanks for replying back, i need the exact behavior shown in demo project i.e. https://msarsha.github.io/ng2-right-click-menu/
It always shows the custom context menu everytime i right click which is what i want, but right now when i right click it shows me custom menu but when i again click then original system menu opens up....so its like two menu are visible at same time....I want the exact behavior shown in demo project....but i am confused how to achieve this?

@msarsha
Copy link
Owner

msarsha commented Apr 11, 2019

Oh, I understand, I though you are right clicking the custom menu.
I will look into implementing this behavior.

@ankitsalu
Copy link
Author

ankitsalu commented Apr 11, 2019

Oh, I understand, I though you are right clicking the custom menu.
I will look into implementing this behavior.

But how is it done on - demo project? .....i think this feature already exists and working but i am not getting which property to set to enable this feature.... https://msarsha.github.io/ng2-right-click-menu/

@msarsha
Copy link
Owner

msarsha commented Apr 11, 2019

The implementation is completely different on the current version (I should also update the demo project) so there is no way to enable it atm.

@ankitsalu
Copy link
Author

The implementation is completely different on the current version (I should also update the demo project) so there is no way to enable it atm.

thanks :-) but are you planning do to this? i need to implement this solution, and not have much time...many thanks for your help

@msarsha
Copy link
Owner

msarsha commented Apr 11, 2019

Hopefully I will have some free time this evening to check it out.
You are also more then welcome to try implementing it and submit a PR - this will be much appreciated.
😄

@ankitsalu
Copy link
Author

Hopefully I will have some free time this evening to check it out.
You are also more then welcome to try implementing it and submit a PR - this will be much appreciated.
😄

thanks a lot :-) i will definitely try 👍

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

No branches or pull requests

2 participants