-
Notifications
You must be signed in to change notification settings - Fork 5
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(web-app): fix page not found vertical spacing #1779
Conversation
Thanks @francinelucca! @aagonzales @kingtraceyj can you review the screenshots here? Should there be space between the two sentences? Or should there just be space around the copy block like here: #1778 |
Design guidance here: https://ibm-studios.slack.com/archives/C06P07KPAUR/p1709913167917949 |
❗ The commits in this PR will result in the following version changes: 🐛 PATCH release: |
Quality Gate passedIssues Measures |
Ready for re-rev |
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
Closes #1778
Changelog
Changed
Testing / reviewing
Verify vertical spacing in lg/sm breakpoints: