-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[email protected] #21363
[email protected] #21363
Conversation
@cchaos Since this is a patch during FF, I'm not adjusting templates to use the new EuiPage stuff. I checked that the old old overwrites (which we used to limit widths) still exist and work. |
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.
Yeah nothing major or breaking here. Visually, fixes some page layouts.
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.
Update LGTM
💚 Build Succeeded |
Conflicts: src/core_plugins/kibana/public/dashboard/listing/__snapshots__/dashboard_listing.test.js.snap
This is a patch release of EUI that targets some popover issues in tables / context menus. It also includes the restrictWidth prop addition in 3.0.2 which targets EuiPage (and which makes up the majority of the snapshot updates).
cc @peteharverson