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 v27.4.0 #74004

Merged
merged 49 commits into from
Aug 14, 2020
Merged

Upgrade EUI to v27.4.0 #74004

merged 49 commits into from
Aug 14, 2020

Conversation

thompsongl
Copy link
Contributor

@thompsongl thompsongl commented Jul 31, 2020

Summary

v26.3.1v27.4.0

  • Changed underlying list virtualization library. Some EuiComboBox and EuiFilter* lists will technically have one fewer item in the DOM on initial load, although no real change occurs because the now-absent item was not visible initially without scrolling.
  • Changed underlying focus trap library. Better screen reader and keyboard navigation experience
  • Exported types from '@elastic/eui/lib/test'

Nearly all non-snapshot changes relate to the first two points. EuiSelectable and EuiComboBox have increased virtualization (in places where the list has virtualized scrolling) which results in fewer items initially selectable by DOM (and Selenium) methods. Also, slight DOM rearrangement resulted in data-test-subj .at()/.eq() index changes.

Snapshot updates are largely related to an internal change to the structure of EuiButton (and variants), resulting in subcomponent naming and className churn. Also, buttons in a loading state (isLoading prop), now render a disabled button to prevent clicks.


27.4.0

  • Added customOptionText prop to EuiComboBox (#3811)

Bug fixes

  • Improve EuiDataGrid Chrome rendering performance in full screen (#3726)
  • Removed @elastic/eui/src-docs entries from published eui.d.ts (#3856)

27.3.1

Bug fixes

  • Fixed bug in all input fields placeholders in Safari that weren't vertically centered (#3809)
  • Removed pointer-events: none in both EuiButton & EuiButtonEmpty to not override the pointer-events: auto in the button mixin euiButtonContentDisabled (#3824)
  • Fixed bug in EuiPagination showing wrong page count when compressed prop is true. (#3827)
  • Fixed bug in EUI's input field components where their inputRef couldn't be a RefObject (#3822)
  • Moved react-view and html-format to be devDependencies (#3828)
  • Fixed EuiComboBox keyboard selection when sortMatchesBy=startsWith (#3823)
  • Fixed EuiCodeEditor not exiting edit mode with esc when enableLiveAutocompletion=true (#3833)
  • Fixed issue where EuiDataGrid's cell expansion popover would sometimes render as a scrollable element (#3832)

27.3.0

  • Added possibility to hide "Rows per page" select in EuiDataGrid (#3700)
  • Updated lodash to v4.17.19 (#3764)
  • Added returnKey glyph to EuiIcon (#3783)
  • Added type prop to EuiFieldPassword to support toggling of obfuscation (#3751)

Bug fixes

  • Fixed bug in EuiDataGrid not calculating the width correctly (#3789)
  • Fixed EuiComboBox marking some very long inputs as invalid (#3797)

27.2.0

  • Added analyzeEvent glyph in EuiIcon (#3729)
  • Updated EuiComboBox to allow the options list to open for single selection custom options (#3706)
  • Added useEuiI18n hook for localization (#3749)
  • Added a hit enter badge to EuiComboBox when focusing an option and for empty states that allow pressing enter (#3782)

Bug fixes

  • Fixed EuiComboBox always showing a scrollbar (#3744)
  • Replaced react-focus-lock with react-focus-on (#3631)
  • Fixed errors in EuiSuperDatePicker related to invalid and null date formatting (#3750)
  • Fixed type definitions for findTestSubject and takeMountedSnapshot (#3763)
  • Fixed EuiComboBox not allowing clicks on previously virtualized items when inside of EuiFormRow (#3784)
  • Removed [Space] as a way to select options in EuiSelectable (#3784)
  • Fixed type definition for windowProps in EuiSelectable (#3787)

27.1.0

  • Added titleElement and descriptionElement props to EuiStat (#3693)
  • Updated securityAnalyticsApp app icon (#3720)
  • Removed src/test and @types/enzyme references from eui.d.ts (#3715)
  • Added index.d.ts file to lib/test and es/test (#3715)
  • Added descriptionFlexItemProps and fieldFlexItemProps props to EuiDescribedFormGroup (#3717)
  • Expanded EuiBasicTable's default action's name configuration to accept a function that returns a React node (#3739)
  • Added internal use only button building blocks for reusability in other button components (#3730)

27.0.0

  • Added paddingSize prop to EuiCard (#3638)
  • Added isClearable and placeholder options to EuiColorPicker (#3689)
  • Added SASS helper files for EUI theme globals (#3691)
  • Add label, labelProps and valueText props to EuiProgress (#3661)

Bug fixes

  • Fixed a bug in EuiResizableContainer preventing nested containers (#3699)
  • Fixed a bug in EuiResizableContainer preventing resizing by arrow keys in some cases (#3699)
  • Fixed EuiHorizontalSteps rendering over EuiHeader (#3707)
  • Fixed bug where EuiSuperSelect lost focus after a value selection (#3734)

Breaking changes

  • Significant accessibility refactor of EuiSelectable (#3169)
    • react-virtualized replaced with react-window
    • virtualizedProps on EuiSelectableOptionsList renamed to windowProps
    • Removed rootId and added makeOptionId, listId, and setActiveOptionIndex to EuiSelectableList
    • Added listId to EuiSelectableSearch
    • options passed into EuiSelectable cannot have an id
    • Requires an onChange to be passed into EuiSelectableSearch

Copy link
Contributor

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

Canvas changes lgtm

Copy link
Contributor

@joshdover joshdover left a comment

Choose a reason for hiding this comment

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

Platform changes LGTM

Copy link
Contributor

@chrisronline chrisronline 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 stack monitoring

Copy link
Contributor

@XavierM XavierM 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 approve this PR ;)

Copy link
Contributor

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

@cauemarcondes cauemarcondes 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 LGTM

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Rewviewed code of Kibana App ownership only. Code LGTM

@thompsongl
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

async chunks size

id value diff baseline
rollup 330.5KB -2.0B 330.5KB
securitySolution 7.2MB +620.0B 7.2MB
watcher 602.9KB -2.0B 602.9KB
total +616.0B

page load bundle size

id value diff baseline
visTypeTimeseries 280.9KB +476.0B 280.5KB

History

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

@thompsongl thompsongl merged commit bbee1f9 into elastic:master Aug 14, 2020
thompsongl added a commit to thompsongl/kibana that referenced this pull request Aug 14, 2020
* eui to 27.1.0

* eui to 27.2.0

* buttoniconside type

* euiselectable type

* update onScroll callback and polyfill size references

* findTestSubject ts

* buttoncontent and collapsiblenav src snapshot updates

* update prop retrieval

* xpack snapshots

* jest updates

* type fixes

* more snapshots

* virtual list changes

* more virtualization changes

* merge

* fix functional tests

* data-test-subj for indexPatter-switcher

* storyshots

* eui to 27.3.1

* Fix unit tests

* Fix broken unit test

* Updated snapshots

* Fixed types

* search for value in euiselectable before selection

* select the correct element

* mock virtualized dep

* ts fix

* reinstate storyshot

* ts fix

Co-authored-by: Chandler Prall <chandler.prall@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
thompsongl added a commit that referenced this pull request Aug 14, 2020
* eui to 27.1.0

* eui to 27.2.0

* buttoniconside type

* euiselectable type

* update onScroll callback and polyfill size references

* findTestSubject ts

* buttoncontent and collapsiblenav src snapshot updates

* update prop retrieval

* xpack snapshots

* jest updates

* type fixes

* more snapshots

* virtual list changes

* more virtualization changes

* merge

* fix functional tests

* data-test-subj for indexPatter-switcher

* storyshots

* eui to 27.3.1

* Fix unit tests

* Fix broken unit test

* Updated snapshots

* Fixed types

* search for value in euiselectable before selection

* select the correct element

* mock virtualized dep

* ts fix

* reinstate storyshot

* ts fix

Co-authored-by: Chandler Prall <chandler.prall@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Chandler Prall <chandler.prall@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EUI Feature:Embedding Embedding content via iFrame 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
Development

Successfully merging this pull request may close these issues.