-
Notifications
You must be signed in to change notification settings - Fork 38
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
Website Docs Visual Separation #1402
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.
Can you use our css variables (--space-1
, --space-2
, etc) instead of magic numbers?
Also can you include "Before" vs "After" screenshots in PR description?
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 found the related padding in this file, maybe try meddling with this number? also make sure all other pages still look ok too |
what problem is this PR actually trying to solve? because the previous spacing looks better to me. |
The spacing between the different sections of the documents are too close together, they need more visual separation The new styling does seperate the sections, but it does need refining to keep the website looking clean and polished. |
i believe that comment was already addressed in #1193 |
OK. I'll close the PR, it seems redundant to try to fix a resolved comment. |
is there some spacing you personally think still needs to be improved? because maybe that PR didn't fully solve the problem. |
The main thing I notice is the lack of space between the For example: |
Co-authored-by: Mayank <[email protected]>
Co-authored-by: Mayank <[email protected]>
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 👍
Changes
#941 - Adds visual separation in between different sections of demos to make them more distinct
Before:
After:
Testing
N/A (yet, maybe)
Docs
N/A (we're just making the docs look cleaner)