-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[Tree] Fix a few regressions #2466
Conversation
llorca
commented
May 4, 2018
- Proper icon color
- Proper popover wrapper/target centering in secondary label
- Chevron icon for animated expand icon
- Improve example
center popover wrappers/targets in secondary labelPreview: documentation | landing | table |
} | ||
|
||
.#{$ns}-icon { | ||
margin: $tree-icon-spacing; | ||
} | ||
} | ||
|
||
#{$icon-classes} { | ||
color: $pt-icon-color; |
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 selector is not scoped!!!! if you actually want to change all icons default colors then this is not the correct file. but i suspect you just want to scope this.
also merge conflicts |
@@ -127,6 +130,12 @@ $tree-icon-spacing: ($tree-row-height - $pt-icon-size-standard) / 2 !default; | |||
padding: 0 ($pt-grid-size / 2); | |||
line-height: $tree-row-height; |
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.
remove this.
merge conflictsPreview: documentation | landing | table |
scope icon colorPreview: documentation | landing | table |
test refactorsPreviews: documentation | landing | table |
test refactorsPreviews: documentation | landing | table |
remove line-heightPreviews: documentation | landing | table |