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
Update timeline at tablet/desktop #11396
Conversation
a9defda
to
6ec8647
Compare
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
Size Change: +502 B (0%) Total Size: 793 kB ℹ️ View Unchanged
|
f81bbf0
to
a8569fd
Compare
a8569fd
to
078db88
Compare
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.
This looks good, better even!
Some thoughts about the interface and props, but none of them blockers and can all be adressed in follow-ups if we chose to.
border-left: 1px solid ${colour}; | ||
border-right: 1px solid ${colour}; |
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.
Would the use of --timeline-event-border
not be appropriate here? It brings about several benefits:
- no dynamic generation of Emotion styles
- consistency across all formats
It might require quite a bit more work we can address as a follow-up, as I appreciate the logic for overrides is a bit finnicky and this may not be the best PR to do this :D
Closing as I misinterpreted designs. We cannot remove the left and right border lines of the article as this would break the contract with the Standard Article design |
What does this change?
Updates the timeline designs at tablet and desktop breakpoints:
Why?
Fixes: #11248
To match designs: https://www.figma.com/file/xPr2tZZger7pszcO075tAQ/New-Formats---April-23?type=design&node-id=4667-8581&mode=design&t=AA51JnDiAuLob1b3-0
Screenshots