-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix(context menu): add cursor pointer to menu host #1662
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1662 +/- ##
==========================================
+ Coverage 83.41% 83.42% +<.01%
==========================================
Files 232 232
Lines 7563 7564 +1
Branches 664 664
==========================================
+ Hits 6309 6310 +1
Misses 1061 1061
Partials 193 193
|
@@ -6,6 +6,15 @@ | |||
@import '../../styles/core/mixins'; | |||
|
|||
@mixin nb-context-menu-theme() { | |||
|
|||
.context-menu-host { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why don't we just put these styles to .component.scss
to :host {}
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's directive
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ops
Please read and mark the following check list before creating a pull request:
Short description of what this resolves:
Fixes #1369