-
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
feat(widgets): turn on viewEncapsulation.None #2784
Conversation
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.
I'm for this change overall:
- need to check what could be impacted a bit more
- +1 for removing /deep/
Noted just one thing that is broken in demo app
@@ -1,4 +1,4 @@ | |||
:host>select { | |||
ngb-datepicker-navigation-select > select { |
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.
this should be more specific without encapsulation, ex:
ngb-datepicker-navigation-select > .custom-select
src/tooltip/tooltip.scss
Outdated
:host { | ||
&.bs-tooltip-top .arrow, | ||
&.bs-tooltip-bottom .arrow { | ||
.bs-tooltip { |
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.
I guess it should be specific to ngb-popover-window
, not to any .bs-tooltip
src/popover/popover.scss
Outdated
:host { | ||
&.bs-popover-top .arrow, | ||
&.bs-popover-bottom .arrow { | ||
.bs-popover { |
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.
I guess it should be specific to ngb-popover-window
, not to any .bs-popover
60ee686
to
547a8d5
Compare
Thanks for the review ! The changes have been done. |
@fbasso I know it's really some extreme nitpicking here, but we should rewrite the commit message whenever merging this PR to turning off ViewEncapsulation It sounds clearer to me from a git history purpose |
547a8d5
to
e9aab53
Compare
e9aab53
to
8d2d2a6
Compare
The purpose of this PR is to allow a better widgets customization with css.