Merged
Conversation
tylersmalley
approved these changes
Aug 11, 2020
alisonelizabeth
approved these changes
Aug 11, 2020
Contributor
alisonelizabeth
left a comment
There was a problem hiding this comment.
ES UI changes LGTM.
poffdeluxe
reviewed
Aug 11, 2020
...ins/canvas/public/components/asset_manager/__stories__/__snapshots__/asset.stories.storyshot
Show resolved
Hide resolved
chrisronline
approved these changes
Aug 12, 2020
Contributor
chrisronline
left a comment
There was a problem hiding this comment.
LGTM for stack monitoring
XavierM
approved these changes
Aug 12, 2020
shahzad31
approved these changes
Aug 13, 2020
Contributor
shahzad31
left a comment
There was a problem hiding this comment.
Uptime changes LGTM !!
Contributor
Author
|
@elasticmachine merge upstream |
timroes
approved these changes
Aug 14, 2020
Contributor
timroes
left a comment
There was a problem hiding this comment.
Rewviewed code of Kibana App ownership only. Code LGTM
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
💚 Build SucceededBuild metricsasync chunks size
page load bundle size
History
To update your PR or re-run it, just comment with: |
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>
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
v26.3.1⏩v27.4.0'@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
classNamechurn. Also, buttons in a loading state (isLoadingprop), now render a disabled button to prevent clicks.27.4.0customOptionTextprop toEuiComboBox(#3811)Bug fixes
EuiDataGridChrome rendering performance in full screen (#3726)@elastic/eui/src-docsentries from published eui.d.ts (#3856)27.3.1Bug fixes
pointer-events: nonein bothEuiButton&EuiButtonEmptyto not override thepointer-events: autoin the button mixineuiButtonContentDisabled(#3824)EuiPaginationshowing wrong page count whencompressedprop is true. (#3827)inputRefcouldn't be aRefObject(#3822)react-viewandhtml-formatto bedevDependencies(#3828)EuiComboBoxkeyboard selection whensortMatchesBy=startsWith(#3823)EuiCodeEditornot exiting edit mode withescwhenenableLiveAutocompletion=true(#3833)EuiDataGrid's cell expansion popover would sometimes render as a scrollable element (#3832)27.3.0EuiDataGrid(#3700)v4.17.19(#3764)returnKeyglyph toEuiIcon(#3783)typeprop toEuiFieldPasswordto support toggling of obfuscation (#3751)Bug fixes
EuiDataGridnot calculating the width correctly (#3789)EuiComboBoxmarking some very long inputs as invalid (#3797)27.2.0analyzeEventglyph inEuiIcon(#3729)EuiComboBoxto allow the options list to open for single selection custom options (#3706)useEuiI18nhook for localization (#3749)EuiComboBoxwhen focusing an option and for empty states that allow pressing enter (#3782)Bug fixes
EuiComboBoxalways showing a scrollbar (#3744)react-focus-lockwithreact-focus-on(#3631)EuiSuperDatePickerrelated to invalid andnulldate formatting (#3750)findTestSubjectandtakeMountedSnapshot(#3763)EuiComboBoxnot allowing clicks on previously virtualized items when inside ofEuiFormRow(#3784)[Space]as a way to select options inEuiSelectable(#3784)windowPropsinEuiSelectable(#3787)27.1.0titleElementanddescriptionElementprops toEuiStat(#3693)securityAnalyticsAppapp icon (#3720)src/testand@types/enzymereferences fromeui.d.ts(#3715)index.d.tsfile tolib/testandes/test(#3715)descriptionFlexItemPropsandfieldFlexItemPropsprops toEuiDescribedFormGroup(#3717)EuiBasicTable's default action's name configuration to accept a function that returns a React node (#3739)27.0.0paddingSizeprop toEuiCard(#3638)isClearableandplaceholderoptions toEuiColorPicker(#3689)label,labelPropsandvalueTextprops toEuiProgress(#3661)Bug fixes
EuiResizableContainerpreventing nested containers (#3699)EuiResizableContainerpreventing resizing by arrow keys in some cases (#3699)EuiHorizontalStepsrendering overEuiHeader(#3707)EuiSuperSelectlost focus after a value selection (#3734)Breaking changes
EuiSelectable(#3169)react-virtualizedreplaced withreact-windowvirtualizedPropsonEuiSelectableOptionsListrenamed towindowPropsrootIdand addedmakeOptionId,listId, andsetActiveOptionIndextoEuiSelectableListlistIdtoEuiSelectableSearchoptionspassed intoEuiSelectablecannot have anidonChangeto be passed intoEuiSelectableSearch