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 v33.0.0 #99382

Merged
merged 46 commits into from
Jun 1, 2021
Merged

Upgrade EUI to v33.0.0 #99382

merged 46 commits into from
Jun 1, 2021

Conversation

thompsongl
Copy link
Contributor

@thompsongl thompsongl commented May 5, 2021

Summary

[email protected][email protected]

  • EuiCodeBlock: Replaced highlight.js with prism.js
    • Content must be passed as a string
    • Some language name changes (e.g., sh is not a supported alias for bash/shell)
    • kbnUiSharedDeps-elastic | 3.3MB | 2.6MB | -761.6KB
  • EuiTable*: description and a11y updates
    • Sort information is not included in screen reader title content (relies on aria-sort instead)
    • title instead includes description content
  • EuiPopover: fixed initialFocus prop
    • Fix will be backported in EUI and 7.13.x

33.0.0

  • Added autoFocus prop and functionality to EuiComboBox (#4772)
  • Added inherit color option to EuiIcon to force two-tone (app) icons to inherit their parent's color (#4760)
  • Updated EuiBetaBadge, EuiBadge, EuiButtonIcon, EuiButtonContent, EuiCallOut, EuiContextMenuItem, EuiListGroupItem icon usage to inherit their parent's color (#4760)
  • Added iconProps prop to EuiListGroupItem (#4760)
  • Added i18ntokens.json to published package (#4771)
  • Replaced highlight.js with prism.js/refractor for code syntax highlighting in EuiCodeBlock (#4638)

Bug fixes

  • Fixed initialFocus prop functionality in EuiPopover (#4768)
  • Fixed description prop in EuiTable(#4754)

Breaking changes

  • Changed some language syntax references in EuiCodeBlock, such as jsx (#4638)
  • Removed ability to parse non-string content in EuiCodeBlock (#4638)

32.3.0

  • Reduced icon size in EuiButtonEmpty of size xs. (#4759)

Bug fixes

  • Fixed missing i18n tokens for EuiFilePicker (#4750)
  • Fixed EuiComboBox to use correct placeholder text color (#4744)

32.2.0

  • Removed MutationObserver fallback from EuiResizeObserver (#4709)

Bug fixes

  • Fixed EuiInMemoryTable pagination prop to update visible items when changed (#4714)
  • Fixed a bug in EuiFilePicker where the HTML input was being shown when disabled (#4738)
  • Fixed inverted asc and desc labels for EuiDataGrid datetime schema (#4733)

@thompsongl thompsongl added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes EUI v7.14.0 labels May 5, 2021
@botelastic botelastic bot added Team:APM All issues that need APM UI Team support Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels May 26, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@@ -47,6 +47,7 @@ TYPES_DEPS = [
"@npm//@types/node",
"@npm//@types/normalize-path",
"@npm//@types/testing-library__jest-dom",
"@npm//resize-observer-polyfill"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a server-side package. It shouldn't depend on resize-observer-polyfill.

Copy link
Contributor

@cjcenizal cjcenizal left a comment

Choose a reason for hiding this comment

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

ES UI test snapshot changes LGTM.

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

@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.

LGTM in Stack Monitoring

Copy link
Contributor

@mshustov mshustov left a comment

Choose a reason for hiding this comment

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

Ok for the changes in Core code, providing we enforce env separation in the future #99382 (comment)

Copy link
Member

@cnasikas cnasikas left a comment

Choose a reason for hiding this comment

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

Security solution changes LGTM!

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

@qn895
Copy link
Member

qn895 commented May 27, 2021

ML changes LGTM

Copy link
Contributor

@poffdeluxe poffdeluxe left a comment

Choose a reason for hiding this comment

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

Presentation changes LGTM

Copy link
Member

@kertal kertal left a comment

Choose a reason for hiding this comment

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

LGTM, Code owner review, checked out, timestamp sorting in data grid is now correct, merci beaucoup! 👍

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine kibanamachine requested a review from a team June 1, 2021 13:54
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1601 1604 +3

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 4.3MB 4.3MB +12.7KB
home 165.0KB 165.0KB +2.0B
ml 5.9MB 5.9MB +45.0B
securitySolution 6.9MB 6.9MB +112.0B
visTypeTimeseries 1.7MB 1.7MB +80.0B
visTypeVega 2.4MB 2.4MB +4.0B
total +12.9KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-css 501.7KB 503.3KB +1.6KB
kbnUiSharedDeps-elastic 3.3MB 2.6MB -761.6KB
kbnUiSharedDeps-js 6.8MB 6.5MB -337.7KB
total -1.1MB
Unknown metric groups

References to deprecated APIs

id before after diff
canvas 29 25 -4
crossClusterReplication 8 6 -2
fleet 22 20 -2
globalSearch 4 2 -2
indexManagement 12 7 -5
infra 256 146 -110
lens 67 45 -22
licensing 18 15 -3
lists 239 236 -3
maps 286 208 -78
ml 121 115 -6
monitoring 109 56 -53
securitySolution 390 346 -44
stackAlerts 101 95 -6
total -340

History

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

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 👍

@thompsongl thompsongl merged commit 38fd8a2 into elastic:master Jun 1, 2021
thompsongl added a commit to thompsongl/kibana that referenced this pull request Jun 1, 2021
* eui to 33.0.0

* resize observer type inclusion - revisit

* src snapshot updates

* x-pack snapshot updates

* table sort test updates

* code block language sh -> bash

* datagrid datetime sort inversion

* types

* kbn-crypto

* refractor yarn resolution

* refractor yarn resolution

* update cypress tests

* url state test

* trial

* Revert "trial"

This reverts commit adc3538.

* trial anomaly timeout

* Revert "trial anomaly timeout"

This reverts commit 9a11711.

* kbn-telemetry-tools

* Change a useMemo to useCallback so the code executes when intended

* Removed no-longer-used import

* exitOrFail already retries for longer than tryForTime

* Wait for loading indicator to disappear

* Intentionally adding `.only`

* Revert .only

* Increase wait time for the ML chart to load

* Remove unused var

* overflow

* chartWidth

Co-authored-by: Kibana Machine <[email protected]>
Co-authored-by: Alejandro Fernández Haro <[email protected]>
Co-authored-by: Chandler Prall <[email protected]>
thompsongl added a commit that referenced this pull request Jun 1, 2021
* eui to 33.0.0

* resize observer type inclusion - revisit

* src snapshot updates

* x-pack snapshot updates

* table sort test updates

* code block language sh -> bash

* datagrid datetime sort inversion

* types

* kbn-crypto

* refractor yarn resolution

* refractor yarn resolution

* update cypress tests

* url state test

* trial

* Revert "trial"

This reverts commit adc3538.

* trial anomaly timeout

* Revert "trial anomaly timeout"

This reverts commit 9a11711.

* kbn-telemetry-tools

* Change a useMemo to useCallback so the code executes when intended

* Removed no-longer-used import

* exitOrFail already retries for longer than tryForTime

* Wait for loading indicator to disappear

* Intentionally adding `.only`

* Revert .only

* Increase wait time for the ML chart to load

* Remove unused var

* overflow

* chartWidth

Co-authored-by: Kibana Machine <[email protected]>
Co-authored-by: Alejandro Fernández Haro <[email protected]>
Co-authored-by: Chandler Prall <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
Co-authored-by: Alejandro Fernández Haro <[email protected]>
Co-authored-by: Chandler Prall <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EUI 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.14.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.