-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
Pull Request Test Coverage Report for Build 1487
💛 - Coveralls |
text-align: center; | ||
max-height: 130px; | ||
/* stylelint-disable plugin/selector-bem-pattern */ | ||
.kubevirt-cluster-overview__health-grid { |
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.
can we update StorageOverview/ocs-health.scss
for consitency? or unite these two together into one file?
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.
@matthewcarleton I've updated snapshots etc to make the build pass. As @suomiy said, can you please update ocs-health.scss
in this PR too ? For other changes I'd suggest to create a new branch and new PR to divide style changes to smaller pieces.
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.
Yes, I can do that. Thanks @suomiy
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.
Sorry this got larger than it should have. I'll create a new PR for the following updates.
just a simple one to test :)