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
[LOOM-1312] [BpkScrollableCalendarGridList] Moved className to existing wrapper div #3429
Conversation
Visit https://backpack.github.io/storybook-prs/3429 to see this build running in a browser. |
@@ -123,7 +123,7 @@ const BpkScrollableCalendarGridList = (props: Props) => { | |||
monthItemHeights[index] || estimatedMonthItemHeight; | |||
|
|||
const rowRenderer = ({ index, style }: { index: number; style: {} }) => ( | |||
<div style={style}> | |||
<div className={getClassName('bpk-scrollable-calendar-grid-list__item')} style={style}> |
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.
testing the class's only styling display: inline-table;
I don't think it has the effect we expect anymore? Perhaps it's safe to remove?
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.
Thanks, removed
Visit https://backpack.github.io/storybook-prs/3429 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3429 to see this build running in a browser. |
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.
Looks good!
Remember to include the following changes:
[KOA-123][BpkButton] Updating the colour
README.md
(If you have created a new component)README.md