-
Notifications
You must be signed in to change notification settings - Fork 8.5k
Update EUI to 110.0.0 #244866
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
base: main
Are you sure you want to change the base?
Update EUI to 110.0.0 #244866
Conversation
- EUI updated the type and the query might be null for text searches
…o "combobox" in DataView list component
💛 Build succeeded, but was flaky
Failed CI StepsTest FailuresMetrics [docs]Async chunks
Page load bundle
History
cc @mgadewoll |
|
Pinging @elastic/eui-team (EUI) |
|
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
|
Pinging @elastic/fleet (Team:Fleet) |
|
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
rmyz
left a comment
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.
obs presentation changes LGTM
sabarasaba
left a comment
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.
kibana management changes lgtm
jillguyonnet
left a comment
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.
Fleet changes LGTM 🚀
|
Pinging @elastic/obs-presentation-team (Team:obs-presentation) |
PhilippeOberti
left a comment
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.
Code review only, LGTM for the @elastic/security-threat-hunting-investigations team
peteharverson
left a comment
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.
ML changes LGTM (tested the 3 tables with noItemsMessage changes).
Dependency updates
@elastic/eui:v109.2.0⏩v110.0.0Changes
messageprop onEuiInMemoryTabletonoItemsMessageEuiSearchBar'sonChangeprop to check for thequeryargument, as it may benullnowcolorprops on implementations ofEuiPageTemplateandEuiPageTemplate.Header(colorprop wasn't applied before either)Package updates
@elastic/euiv110.0.0EuiSuperDatePickerto expose plain textutcOffsetandtimeZoneNameintimeZoneDisplayProps.customRenderrender function (#9245)EuiHeaderLogoto add a hover style (#9240)EuiQuickSelectPanelavailable for importing from the@elastic/euipackage (#9239)EuiAvatarto support Emoji Sequence ("advanced") in theinitialsprop (#9227)colorprop onEuiPageHeaderto supporttransparent(default) andplainbackgrounds. (#9220)colorprop onEuiPageto supporttransparent(default) andplainbackgrounds. (#9220)EuiPageTemplateto ensurepanelled=truerenders aEuiPageHeaderwith a plain background by default (#9220)EuiPageTemplate's outer wrapper (#9220)Bug fixes
EuiSuperDatePicker's time window buttons whencompressed={true}(#9245)EuiIconvisibility issue withlogoElasticwhencoloris set totextorghostin light mode (#9247)EuiInMemoryTablesupport for controlled search for plain text (whensearchFormat="text") by properly handlingsearch.queryandsearch.defaulQuery(#9142)Breaking changes
messageprop fromEuiInMemoryTable, usenoItemsMessageinstead (#9234)Accessibility
EuiSuperDatePickerby properly labeling them (#9239)EuiSelectableby removing emptyaria-activedescendantattribute when no option is active to ensure the search input is perceivable by screen readers (#9223)