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

Upgrade EUI to v28.2.0 #75700

Merged
merged 27 commits into from
Aug 31, 2020
Merged

Upgrade EUI to v28.2.0 #75700

merged 27 commits into from
Aug 31, 2020

Conversation

thompsongl
Copy link
Contributor

Summary

v27.4.1v28.2.0

  • Support for @elastic/[email protected]
  • New pluggable EuiMarkdownEditor and EuiMarkdownFormat components
  • New EuiSelectableTemplateSitewide component for global search

28.2.0

  • Added EuiSelectableTemplateSitewide as wrapper of EuiSelectable for Elastic's global search component (#3800)
  • Updated styles of EuiMark to override browser default (#3800)
  • Updated EuiHighlight to use EuiMark as matching element wrapper (#3800)
  • Enhanced EuiSelectable's option type to allow for a separate searchableLabel and any generic keys (#3800)
  • Added listProps.onFocusBadge, loadingMessage, noMatchesMessage, and emptyMessage props to EuiSelectable (#3800)
  • Added bordered prop to EuiSelectableMessage (#3800)

Bug fixes

  • Fixed display of EuiBadge if only the iconType is passed (#3800)
  • Fixed accessibility error in EuiSelectable when the list isn't on the page (#3800)
  • Fixed accessibility error in EuiPopover when ownFocus = false (#3800)

28.1.0

  • Added isLoading and isLoadingMessage props to EuiAccordion (#3879)
  • Added testenv mock for EuiFocusTrap (#3930)

Bug fixes

  • Fixed bug in EuiCodeBlock content overlapping with control buttons when whiteSpace was set to "pre" (#3853)
  • Fixed EuiFocusTrap not applying provided style prop (#3916)
  • Fixed bug in EuiDataGrid when a new pagination object would cause every cell to render (#3919)

28.0.0

  • Update createTheme to apply latest changes to elastic charts Theme. (#3792)
  • Added icons for appSearchApp and workplaceSearchApp to EuiIcon (#3859)
  • Added unlink glyph to EuiIcon (#3869)
  • Added EuiMarkdownEditor and EuiMarkdownFormat components (#3522)

Bug fixes

  • Fixed EuiFacetGroup container expansion due to negative margin value (#3871)
  • Fixed EuiComboBox delimeter-separated option creation and empty state prompt text (#3841)
  • Fixed EuiDataGrid not properly resizing within a fixed height container (#3894)
  • Fixed bug in EuiFieldPassword where an edge case mutated its append prop (#3884)

Breaking changes

  • Requires @elastic/charts version 20.0.0 and above for chart theming utils.

@thompsongl thompsongl added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Aug 21, 2020
@nickofthyme
Copy link
Contributor

Charts updates include

fix #73167
close #75031

@nickofthyme
Copy link
Contributor

@thompsongl it looks like the failing test (Security solution Cypress Tests) is from the search_bar.spec.ts. Let me know if this is due to my changes and I can look into it.

@thompsongl
Copy link
Contributor Author

Let me know if this is due to my changes and I can look into it.

Unrelated. I'll take on fixing it.

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

@thompsongl thompsongl marked this pull request as ready for review August 25, 2020 21:13
@thompsongl thompsongl requested a review from a team as a code owner August 25, 2020 21:13
Copy link
Contributor

@smith smith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APM changes look good!

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maps changes LGTM

Copy link
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for platform changes

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@wylieconlon wylieconlon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and reviewed the changes to Lens, LGTM.

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@crob611 crob611 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Canvas just has a snapshot update and it looks good to me. 👍

Copy link
Contributor

@igoristic igoristic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stack Monitoring looks good 👍

Copy link
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we have a boolean flipped to false instead of true.

Co-authored-by: Justin Kambic <[email protected]>
Copy link
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uptime changes LGTM!

Copy link
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kibana App team Code review LGTM, I also checked the bug on TSVB and it is fixed 🎉 Lens settings panel works as expected and is much better now 👏

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
canvas 1200 -3 1203

async chunks size

id value diff baseline
apm 4.7MB +552.0B 4.7MB
canvas 1.4MB -1.3KB 1.4MB
enterpriseSearch 358.5KB +89.0B 358.4KB
maps 3.3MB +40.0B 3.3MB
observability 296.9KB +15.0B 296.9KB
securitySolution 9.5MB +736.0B 9.5MB
visTypeTimeseries 1.8MB +1.2KB 1.8MB
total +1.3KB

page load bundle size

id value diff baseline
lens 867.3KB +93.0B 867.2KB
upgradeAssistant 64.8KB +89.0B 64.7KB
total +182.0B

oss distributable file count

id value diff baseline
total 27303 +1057 26246

distributable file count

id value diff baseline
total 45423 +1055 44368

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@thompsongl thompsongl merged commit ee73c59 into elastic:master Aug 31, 2020
thompsongl added a commit to thompsongl/kibana that referenced this pull request Aug 31, 2020
* eui to v28.0.0

* eui to 28.2.0

* euiselectableoptions type update

* targeted nohoist

* src snapshot updates

* x-pack snapshot updates

* strong -> mark

* upgrade @elastic/charts to v21.0.1

* strong -> mark

* fix charts version merge

* maps -> add_field_tooltip_popover type update

* snapshot update

* Fix gridline visibility

Co-authored-by: Justin Kambic <[email protected]>

Co-authored-by: nickofthyme <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Justin Kambic <[email protected]>
thompsongl added a commit that referenced this pull request Aug 31, 2020
* eui to v28.0.0

* eui to 28.2.0

* euiselectableoptions type update

* targeted nohoist

* src snapshot updates

* x-pack snapshot updates

* strong -> mark

* upgrade @elastic/charts to v21.0.1

* strong -> mark

* fix charts version merge

* maps -> add_field_tooltip_popover type update

* snapshot update

* Fix gridline visibility

Co-authored-by: Justin Kambic <[email protected]>

Co-authored-by: nickofthyme <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Justin Kambic <[email protected]>

Co-authored-by: nickofthyme <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Justin Kambic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.10.0 v8.0.0
Projects
None yet