-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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 v26.3.1 #70243
Upgrade EUI to v26.3.1 #70243
Conversation
Update: Waiting on resolution of #70383 |
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.
ES UI changes look good. I tested the Console history, and the Ingest Node Pipelines inline editor and processor movement-canceling UX. I couldn't test the changes to use_ui_ace_keyboard_mode.ts
since a regression (#71157) means that this functionality currently isn't working on master. Test changes look reasonable but I didn't dig deeply. I assume if they're passing then the changes are fine.
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.
LGTM for platform changes
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 owner changes lgtm
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.
KibanaApp owned Code LGTM 👍 , tested locally in Chrome, Firefox, Safari, fixes the issue described in #6990, scrolling in Firefox over elements with Tooltips is no longer causing the jumping back to top
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.
For Security Solutions
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.
maps/*
code review. LGTM. Thx!
@elasticmachine merge upstream |
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.
LGTM... thanks for updating the storyshot, too!
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.
Design pinged due to change in import path.
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.
uptime changes LGTM !!
|
@elasticmachine merge upstream |
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
* eui to 26.1.0 * Breadcrumb -> EuiBreadcrumb * keyCode -> key * src snapshot updates * update header variables file location * keyCode -> key in tests * x-pack simple snapshot updates * euitooltip updates * euioverlaymask updates * testbed keycode -> key * fixed header sass mixin * euidatagrid test workaround * euioverlapmask mock * euipaginationbutton markup * nav snapshot update * eui to 26.3.0 * flyout z-index fix * euiflyout snapshot updates * style snapshot updates * table actions have machine readable text * table actions have machine readable text * core api docs update * table actions snapshot update * eui to 26.3.1 * update table actions type * mock issue links; timeout done * breadcrumb -> euibreadcrumb Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> # Conflicts: # x-pack/plugins/index_management/__jest__/client_integration/home/index_templates_tab.test.ts # x-pack/plugins/security_solution/public/resolver/view/panels/panel_content_utilities.tsx
* Upgrade EUI to v26.3.1 (#70243) * eui to 26.1.0 * Breadcrumb -> EuiBreadcrumb * keyCode -> key * src snapshot updates * update header variables file location * keyCode -> key in tests * x-pack simple snapshot updates * euitooltip updates * euioverlaymask updates * testbed keycode -> key * fixed header sass mixin * euidatagrid test workaround * euioverlapmask mock * euipaginationbutton markup * nav snapshot update * eui to 26.3.0 * flyout z-index fix * euiflyout snapshot updates * style snapshot updates * table actions have machine readable text * table actions have machine readable text * core api docs update * table actions snapshot update * eui to 26.3.1 * update table actions type * mock issue links; timeout done * breadcrumb -> euibreadcrumb Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> # Conflicts: # x-pack/plugins/index_management/__jest__/client_integration/home/index_templates_tab.test.ts # x-pack/plugins/security_solution/public/resolver/view/panels/panel_content_utilities.tsx * lint fix
…11y-overlay * 'master' of github.com:elastic/kibana: (33 commits) address index templates feedback (elastic#71353) Upgrade EUI to v26.3.1 (elastic#70243) [build] Creates Linux aarch64 archive (elastic#69165) [SIEM][Detection Engine] Fixes skipped tests (elastic#71347) [SIEM][Detection Engine][Lists] Adds read_privileges route for lists and list items [kbn/optimizer] implement "requiredBundles" property of KP plugins (elastic#70911) [Security Solution][Exceptions] - Exceptions modal pt 2 (elastic#70886) [ML] DF Analytics: stop status polling when job stopped (elastic#71159) [SIEM][CASE] IBM Resilient Connector (elastic#66385) jenkins_xpack_saved_objects_field_metrics.sh expects to be run from the KIBANA_DIR in CI Deduplication of entries and items before sending to endpoint (elastic#71297) [services/remote/webdriver] fix eslint error (elastic#71346) send slack notifications on visual baseline failures fix visual regression job (elastic#70999) [Ingest Manager] Add schema to usageCollector. (elastic#71219) [ftr] use typed chromeOptions object, adding TEST_BROWSER_BINARY_PATH (elastic#71279) [Ingest Manager] Fix limited packages incorrect response (elastic#71292) Support multiple features declaring same properties (elastic#71106) [Security_Solution][Resolver]Add beta badge to Resolver panel (elastic#71183) [DOCS] Clarify trial subscription levels (elastic#70636) ...
* master: (78 commits) Bump lodash package version (elastic#71392) refactor: 💡 use allow-list in AppArch codebase (elastic#71400) improve bugfix 7198 test stability (elastic#71250) [Security Solution][Ingest Manager][Endpoint] Optional ingest manager (elastic#71198) [Metrics UI] Round metric threshold time buckets to nearest unit (elastic#71172) [Security Solution][Endpoint] Policy creation callback fixes + Improved error handling in user manifest loop (elastic#71269) [Security Solution] Allow to configure Event Renderers settings (elastic#69693) Fix a11y keyboard overlay (elastic#71214) [APM] UI text updates (elastic#71333) [Logs UI] Limit `extendDatemath` to valid ranges (elastic#71113) [SIEM] fix tooltip of notes (elastic#71342) address index templates feedback (elastic#71353) Upgrade EUI to v26.3.1 (elastic#70243) [build] Creates Linux aarch64 archive (elastic#69165) [SIEM][Detection Engine] Fixes skipped tests (elastic#71347) [SIEM][Detection Engine][Lists] Adds read_privileges route for lists and list items [kbn/optimizer] implement "requiredBundles" property of KP plugins (elastic#70911) [Security Solution][Exceptions] - Exceptions modal pt 2 (elastic#70886) [ML] DF Analytics: stop status polling when job stopped (elastic#71159) [SIEM][CASE] IBM Resilient Connector (elastic#66385) ...
Summary
v24.1.0
⏩v26.3.1
KeyboardEvent.keyCode
(deprecated) withKeyboardEvent.key
Query
/EuiSearchBar
to allow any character inside double-quoted phrasesEuiToolTip
during frequent mouesover/mouseout events26.3.1
isClearable
andplaceholder
options toEuiColorPicker
(#3689)26.3.0
EuiBasicTable
's default action's name configuration to accept any React node (#3688)26.2.0
background.color
toEUI_CHARTS_THEME_LIGHT/DARK.theme
(#3669)gutterSize
prop toEuiFacetGroup
(#3639)EuiCode
andEuiCodeBlock
to reflect only functional props (#3647)EuiResizableContainer
onPanelWidthChange
callback method to include all panel widths (#3630)Query
/EuiSearchBar
to allow any character inside double-quoted phrases (#3432)headerZindexLocation
prop toEuiOverlayMask
(#3655)maskProps
prop toEuiFlyout
andEuiCollapsibleNav
(#3655)Bug fixes
EuiContextMenu
panelonAnimationEnd
transition bug in Chrome (#3656)EuiSkipLink
interactive props and Safari click issue (#3665)z-index
issues withEuiHeader
,EuiFlyout
, and other portal content (#3655)color
prop error inEuiBadge
to be more flexible with what format it accepts (#3655)EuiSuperSelect
popover from moving 16px horizontally when it's close to a window edge (#3685)Theme: Amsterdam
EuiHeaderBreadcrumb
height,onClick
, border-radius, and single item display (#3655)26.1.0
EuiCardProps
andEuiCheckableCardProps
types (#3640)26.0.1
Bug fixes
EuiCode
(#3633)26.0.0
useEuiTextDiff
react hook utility (#3288)EuiOverlayMask
to be a React functional component (#3555)responsive
andmax
behavior ofEuiBreadcrumbs
to always display collapsed items in popover #3578)BREAKPOINTS
andgetBreakpoint
utilities #3578)'any'
option to thestep
prop of theEuiFieldNumber
(#3562)EuiHeader
SASS variables toglobal_styles
(#3592)side
prop toEuiGlobalToastList
for choosing which window side to display toasts (#3600)titleSize
get's implicitly set to 'm' forEuiEmptyPrompt
(#3598)logoElastic
to meet brand guidelines (#3613)EuiStat
(#3617)CommonProps
inEuiColorPalettePickerPaletteTextProps
,EuiColorPalettePickerPaletteFixedProps
andEuiColorPalettePickerPaletteGradientProps
types (#3616)onToggle
callback inEuiAccordion
to allow for external state control (#3614)Bug fixes
display
prop toEuiDataGridColumnSortingDraggable
to passdisplayAsText
prop correctly to the column sorting popover.(#3574)EuiCodeBlockImpl
testenv mock pass-through ofdata-test-subj
attribute (#3560)EuiOverlayMask
by using lifecycle methods (#3555)EuiComboBox
's options listzIndex
positioning when nested in otherzIndex
contexts (#3551)euiHeaderAffordForFixed
mixin's use of header SASS variable (#3592)onClick
as a valid prop forEuiControlBar
icon controls (#3581)EuiToolTip
during frequent mouesover/mouseout events (#3596)EuiBasicTable
custom actions popover from remaining open after click (#3619)Breaking changes
breadcrumb
TS type exported name fromBreadcrumb
toEuiBreadcrumb
(#3578)$euiZComboBox
SCSS variable (value was 8001) (#3551)Theme: Amsterdam
EuiCallout
by removing left border, adding border radius and increasing font weight on titles (#3557)EuiHeaderBreadcrumbs
style to be more prominent (#3578)EuiFilterGroup
border-radius
(#3591)EuiCodeBlock
inline code style to use border radius (#3599)25.0.0
EuiCallOut
to avoid usage of additional space caused by the rendered<div>
element (#3549)invalidCallout
prop toEuiForm
to allow conditional rendering of error callout(#3585)Bug fixes
EuiCard
image corners to be contained within border radius (#3556)EuiKeyPadMenu
andEuiKeyPadMenuItem
aria roles (#3502)EuiFieldSearch
input clear button doesn't show when external input is passed(#3497)EuiBasicTable
footers to always use a uniquekey
(#3559)EuiInMemoryTable
by changing thegetDerivedStateFromProps
to not block the updates as soon as it hits a true if condition (#3579)Breaking changes
EuiHeader
no longer automatically padding directly to the<body>
element (#3538)EuiPagination
,EuiDataGrid
,EuiBasicTable
andEuiInMemoryTable
accessibility, causingEuiPaginationButton
to require a new proppageIndex
(#3294)KeyboardEvent.keyCode
(deprecated) withKeyboardEvent.key
. From@elastic/eui/lib/services
,keyCodes
has been replaced withkeys
, as hascascadingMenuKeyCodes
->cascadingMenuKeys
, andcomboBoxKeyCodes
->comboBoxKeys
. The implementation of all of those exports (as well asaccessibleClickKeys
) all now useKeyboardEvent.key
values. (#3517)