-
Notifications
You must be signed in to change notification settings - Fork 157
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
fix (structured-list): Enable custom max-cols values #11804
fix (structured-list): Enable custom max-cols values #11804
Conversation
Deploy preview created for package Built with commit: 9aa3f261e7b6eca61fbd6a33a788faa93a482e18 |
Deploy preview created for package Built with commit: 9aa3f261e7b6eca61fbd6a33a788faa93a482e18 |
Deploy preview created for package Built with commit: 9aa3f261e7b6eca61fbd6a33a788faa93a482e18 |
Deploy preview created for package Built with commit: 9aa3f261e7b6eca61fbd6a33a788faa93a482e18 |
Deploy preview created for package Built with commit: 9aa3f261e7b6eca61fbd6a33a788faa93a482e18 |
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.
LGTM!
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 to me!
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 to me as well. Thanks for a great PR description. Logic is solid.
Related Ticket(s)
https://jsw.ibm.com/browse/ADCMS-5014
Description
This PR moves the
--max-cols
definitions to the top-levelstructured-list
andpricing-table
components. Previously, each row element set this value, leading tolg
breakpoints always calculating against an assumed value of 16-colsThis change was needed to make the table work more intuitively in situations where it isn't displayed in a full 16-column container, for example within the
table-of-contents
component.Before this PR, a pricing table 12-cols wide, with three table-columns each set to be 4-cols wide would set each table-column to be 25% width (4/16).
With this PR, the pricing table can set
--max-cols: 12
and have those same table-columns set to 33.333% width (4/12)Changelog
Changed
--max-col
values in structured-list and pricing-table components