-
Notifications
You must be signed in to change notification settings - Fork 0
docs: 💄 prettify the guide list in the landing page #199
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
Conversation
joelostblom
left a comment
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 like this format better than the multi-column grid! I was actually going to suggest removing it altogether because I found it redundant with the ToC on the left, but the change here with the additional description adds some value and makes it easier to see that this is the same as the ToC because the order and formatting is the same (ie one vertical column)
signekb
left a comment
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.
Nice! 💄 A comment:
joelostblom
left a comment
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!
Description
Makes it a list of items, rather than a row of them. Adds description to fill in the space a bit.
Needs a quick review.
Checklist
just run-all