-
Notifications
You must be signed in to change notification settings - Fork 844
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
[EuiDescriptionList] New gutterSize
prop
#6175
Conversation
…on-list/gutterSize Merging in the latest code from main
…for the Title and Description based on the size passed into the component
…on-list/gutterSize Pulling in code from main
Preview documentation changes for this PR: https://eui.elastic.co/pr_6175/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_6175/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_6175/ |
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!
Just one suggestion
src/components/description_list/description_list_description.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Greg Thompson <thompson.glowe@gmail.com>
I don't believe so because the prop is intended to be for vertical spacing only, but may I ask why it should?
Possibly, but I think it would affect current iterations of |
Preview documentation changes for this PR: https://eui.elastic.co/pr_6175/ |
That's not my personal intepretation of "gutter" - when I read "gutterSize", I would assume it applies the sizing to both horizontal and vertical gutters, not just vertical. That's how
Same, but in that case, let's combine the |
Ok, gotcha. So as far as current take aways:
|
I'd hang tight on 1. to see if @miukimiu thinks we're OK to apply the size change to the horizontal spacing as well as vertical, but you should be good to move ahead on 2.! |
…criptionList example for the gutterSize, align, and compressed props.
…hall/eui into description-list/gutterSize Pulling in latest code from description-list/gutterSize
I updated the Centered & Compressed example to be Customizing Appearance. I've also added toggles for the |
Preview documentation changes for this PR: https://eui.elastic.co/pr_6175/ |
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.
🎉 🎉 Love the UX for the new docs!! I'll open a follow-up issue for @miukimiu to consider when she gets back about gutterSize
and the horizontal gutter.
Summary
Fixes #5834
Creates the
gutterSize
prop forEuiDescriptionList
which is responsible for adjusting vertical spacing between Description List elements.gutterSize
accepts the t-shirt sizess
andm
(default) to add a top margin to Description List elements. When nogutterSize
is provided, sizem
is used by default.Small
gutterSize
Medium
gutterSize
Checklist