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

Update contextMenu.component.ts #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update contextMenu.component.ts #63

wants to merge 1 commit into from

Conversation

neomagister
Copy link

Fixed menu position for Angular2+

@neomagister
Copy link
Author

neomagister commented Sep 12, 2017

I'm having an issue with position of context menu. I found a solution, please have a look if is ok. Have you experienced similar behaviour? or I'm doing something wrong?

@isaacplmann
Copy link
Owner

Probably half of the issues that I've resolved relate to positioning. I like that your solution makes the code a lot simpler, but I have to check a bunch of edge cases.

Can you add some sample code in the demo that reproduces the problem you're having?

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

Successfully merging this pull request may close these issues.

2 participants