-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
overlay directive #879
Comments
Think that should be |
Hey, i am a student and a little new to material 2. I want to become a regular contributor because angular is so awesome :) I would like to work on this issue, if someone is already working on that please let me know . Any advice on how to go about it is also appreciated :) Thank you |
@jmurphzyo I can't find anything wrong with the plunker demo. Looks like overlay-lay directive has a typo but did something go wrong because of it? |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug, feature request, or proposal:
bug
What is the expected behavior?
n/a
What is the current behavior?
Possible typo
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/o077B6uEiiIgkC0S06dd
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, browsers are affected?
Material2 latest
Is there anything else we should know?
n/a
I believe there is a typo here https://github.com/angular/material2/blob/master/src/core/overlay/overlay-directives.ts#L91
originX is set to overlayX position. I don't know if that was intended.
The text was updated successfully, but these errors were encountered: