Skip to content
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] Add columnGap prop for configurable column spacing #7011

Closed
breehall opened this issue Jul 28, 2023 · 4 comments
Closed
Assignees
Labels
design decision feature request good first issue help wanted The EUI team is looking for community members to pick up and implement this issue stale-issue stale-issue-closed

Comments

@breehall
Copy link
Contributor

Is your feature request related to a problem? Please describe.
EuiDescriptionList columns currently default to a width of 50%. Depending on the screen size, these columns can be spread quite far apart. The new prop would make this value configurable.

Describe the solution you'd like
A new columnGap prop that accepts s and m (default) as values to control the column gap. m would be the default and would be 50% (the amount used today).

Describe alternatives you've considered
The alternatives to this are kind of hacky. One such hack is using the different description list components and setting a class on DescriptionListTitle with class setting the flex-basis to the desired max width of the title column.

CC: @kyrspl

image

@cee-chen cee-chen added good first issue help wanted The EUI team is looking for community members to pick up and implement this issue labels Aug 7, 2023
@github-actions
Copy link

github-actions bot commented Aug 7, 2023

👋 Thank you for your suggestion or request! While the EUI team agrees that it's valid, it's unlikely that we will prioritize this issue on our roadmap. We'll leave the issue open if you or anyone else in the community wants to implement it by contributing to EUI. If not, this issue will auto close in one year.

@JasonStoltz
Copy link
Member

@kyrspl We've identified this as a lower priority for the EUI team. However, we would welcome a contribution. We've labeled this as a "good first issue" and "help wanted" in the meantime.

Copy link

github-actions bot commented Feb 4, 2024

👋 Hi there - this issue hasn't had any activity in 6 months. If the EUI team has not explicitly expressed that this is something on our roadmap, it's unlikely that we'll pick this issue up. We would sincerely appreciate a PR/community contribution if this is something that matters to you! If not, and there is no further activity on this issue for another 6 months (i.e. it's stale for over a year), the issue will be auto-closed.

Copy link

github-actions bot commented Aug 2, 2024

❌ Per our previous message, this issue is auto-closing after having been open and inactive for a year. If you strongly feel this is still a high-priority issue, or are interested in contributing, please leave a comment or open a new issue linking to this one for context.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design decision feature request good first issue help wanted The EUI team is looking for community members to pick up and implement this issue stale-issue stale-issue-closed
Projects
None yet
Development

No branches or pull requests

4 participants