fix: Clean up chip list spacing #164
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@conrad-vanl assigning you because nathan is ooo |
| margin-left: 1rem; | ||
| } | ||
| * > &:last-child { | ||
| margin-left: 1rem; | ||
| & > *:last-child { |
There was a problem hiding this comment.
I wrote this CSS in the previous PR, had it working, then moved it to a different section and butchered it when re-writing 😅
|
Never mind, I reviewed below 👇 |
| * > &:first-child { | ||
| .chip-list-feature ul { | ||
| overflow-x: scroll; | ||
|
|
There was a problem hiding this comment.
I think adding padding-bottom: 1rem; here might be nice.
There was a problem hiding this comment.
Looks good without the scrollbar, looks not so good with the scrollbar. Margin might look great.
There was a problem hiding this comment.
Splitting 1rem across margin + padding
Just margin
I think Iike splitting the most
There was a problem hiding this comment.
Yeah Splitting works for me
Co-authored-by: Nathan Lewis <[email protected]>
Thanks! what browser did you see that on? |
Arc and Chrome |
https://3.basecamp.com/3926363/buckets/32694519/card_tables/cards/7057831230#__recording_7062942883
✏️ Solution
🔬 To Test
on
web-embeds📸 Screenshots