-
Notifications
You must be signed in to change notification settings - Fork 175
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
[JSX] Add CSSGrid component type #6090
Conversation
This adds a CSSGrid component type. A CSSGrid aligns a number of cards (passed as a prop) in a grid. The component takes care of converting the values passed to it into cards with an appropriate title, size and positioning.
@driusan i am wondering why CSSGrid over Flexbox? |
@zaliqarosli flexbox only works in 1 dimension. A grid is like a 2 dimensional flexbox. |
I'd say "see https://www.webdesignerdepot.com/2018/09/grid-vs-flexbox-which-should-you-choose/" but the images in that link seem to be broken.. |
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.
besides these aesthetic requests, code looks good!
Co-Authored-By: Zaliqa <zaliqa.rosli@mcin.ca>
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.
oops duplicate because I requested the changes on only the one line
Co-Authored-By: Zaliqa <zaliqa.rosli@mcin.ca>
This adds a CSSGrid component type. A CSSGrid aligns
a number of cards (passed as a prop) in a grid. The
component takes care of converting the values passed
to it into cards with an appropriate title, size
and positioning.