-
Notifications
You must be signed in to change notification settings - Fork 3
Grid: Center justify text content #218
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
❌ Playwright visual snapshot differences were detected.View the Playwright report |
/approve-snapshots |
Updating snapshots. Click here to see the status. |
Please download the artifacts here and commit your updated screenshots. |
f362d0b
to
4af59e8
Compare
This messes up the breadcrumb alignment. I think it'll look better if the crumbs follow the content. |
4af59e8
to
5a2954a
Compare
❌ Playwright visual snapshot differences were detected.View the Playwright report |
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 locally
a21d084
to
7925672
Compare
❌ Playwright visual snapshot differences were detected.View the Playwright report |
7925672
to
3af99ff
Compare
❌ Playwright visual snapshot differences were detected.View the Playwright report |
Move breadcrumb into same element as text content to allow it to follow the content on larger screens. Slightly increase gutter gap on larger screens.
3af99ff
to
54a837c
Compare
Center justify text content. This matches responsive behaviour similar to stripe docs.
Screen.Recording.2025-04-15.at.11.38.39.mov