-
Notifications
You must be signed in to change notification settings - Fork 37
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
STCOM-1408: Add marginTop0 class to MessageBanner component #2425
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 lost here -- I see the prop and the style, but not where anything actually happens with that prop. How does this work?
Sorry my question wasn't clear, @mariia-aloshyna. What I meant was, I see this new prop on line 49, but it's never applied anywhere. The remaining props are collected into |
eadba03
to
b059e84
Compare
No magic, I just missed a piece of code 🤦♀️. Updated the PR, thank you for catching it, now it works as expected. |
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.
No worries, PR review working exactly like it supposed to 🤝
How did that test ever pass though? 🤔
|
Introduce the marginTop0 class to the MessageBanner component to remove the default top margin. ## Refs https://folio-org.atlassian.net/browse/STCOM-1408
Introduce the marginTop0 class to the MessageBanner component to remove the default top margin.
Refs
https://folio-org.atlassian.net/browse/STCOM-1408