-
Notifications
You must be signed in to change notification settings - Fork 829
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
[EuiFlexGrid] Add justifyContent
and alignItems
like EuiFlexGroup
#4878
Comments
@cchaos Seeing as |
We've stayed away from adding those because it's a lot to maintain and then |
I think we should close this issue. |
The issue is still valid for EuiFlexGrid, just not EuiFlexItem. |
I am on it! |
@cchaos FlexGroup and FlexGrid are incredibly similar, AFAICT grid realistically should just be a group with an implicit Grid is also missing reverse directions, as well a check on the component prop (group's is limited to div/span). A breaking change to unify them and deprecate grid is probably a little OTT, but maybe grid should just use group under the hood? |
👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment. |
❌ We're automatically closing this issue due to lack of activity. Please comment if you feel this was done in error. |
Re-opening. Still low-hanging fruit. Most likely to get tackled during the CSS-in-JS conversion. |
EuiFlexGroup
hasjustifyContent
andalignItems
props, butEuiFlexGrid
surprisingly doesn't.The text was updated successfully, but these errors were encountered: