-
Notifications
You must be signed in to change notification settings - Fork 258
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
More emotionjs migration changes #4362
Conversation
margin-bottom: 20px; | ||
width: 100%; | ||
|
||
& > li .hover-button { |
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.
@joeyorlando I'm migrating more code to emotion and have come across this part, can't really figure out where's hover-button
coming from as it's not being defined in any of the add responder styles. Do you happen to know where is that coming from? I know you've been working on this stuff
The only hover-button
I see is part of the integration display within the routes, but that shouldn't be related to add responders in any way
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.
great job!
grafana-plugin/src/containers/AddResponders/__snapshots__/AddResponders.test.tsx.snap
Show resolved
Hide resolved
1085f45
to
540ac1c
Compare
border: ${theme.colors.border.weak}; | ||
display: flex; | ||
flex-direction: column; | ||
border-radius: ${theme.shape.radius.default}; |
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.
With grafana 9.4 themes, shape.radius
is not defined.
Could you return to fixed value, thanks.
What this PR does
vars.css
, specifically tags-related, but not only