-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
CSS style rule updates for issue #5516 #5526
base: master
Are you sure you want to change the base?
CSS style rule updates for issue #5516 #5526
Conversation
🚀 Preview for commit 2d2ef1a at: https://663ba19250d9050e48a1cde8--layer5.netlify.app |
Thank you, @JonathanKoerber 👏 |
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.
Unwanted margins on other sistent pages. Please fix @Jonathan-Foo
The ss is from guidance page of color
Hey @JonathanKoerber , let's discuss this on today's website meeting at 12:00 pm GMT / 5:30 PM IST. I am adding this as an agenda item. Please join if you are available during the meet. |
Thanks for the feedback. I'll take another look at this issue. |
@JonathanKoerber thanks, let us know when you fix this |
Screencast.from.2024-05-20.18-49-34.webmCan you work on this much unnecessary white space |
I'll work on the white space issue. Thanks for the feedback. |
e61d8b1
to
b6d3406
Compare
🚀 Preview for commit b6d3406 at: https://664fd71eaf6b27a2792c7a3a--layer5.netlify.app |
@JonathanKoerber merge conflicts... |
…sed win identiy section page Signed-off-by: JonathanKoerber <jonathankoerber@gmail.com>
9fd8fba
to
3181d43
Compare
🚀 Preview for commit a9d5b96 at: https://6653d4bb4b92a6fe2f91a1a9--layer5.netlify.app |
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.
A small item, @JonathanKoerber: a couple of new merge conflicts cropped up.
Thanks for seeing this through. It's coming along nicely.
The SistentWrappre page section added margin-top: 15 to extend the container up. The margin-top in h2 and h3 was changed from -7 to 7, bringing text back inside the parent container and aligning the infra-page component with the top section.
JoinCommunityWrapper width 18rem => 25rem to match the current website. g
Description
This PR fixes #5516
Notes for Reviewers
Signed commits