-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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 v30.1.1 #81499
Upgrade EUI to v30.1.1 #81499
Conversation
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 monitoring
!
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.
Tested and LGTM - left one small a11y I'm not sure about. It's probably nothing, just wanted to make sure it was considered.
src/plugins/vis_default_editor/public/components/sidebar/controls.tsx
Outdated
Show resolved
Hide resolved
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.
Presentation changes look good to me
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 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.
🚀
x-pack/plugins/security_solution/public/cases/components/case_status/index.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/cases/components/case_view/index.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/cases/components/case_view/index.tsx
Outdated
Show resolved
Hide resolved
💚 Build SucceededMetrics [docs]@kbn/optimizer bundle module count
@kbn/ui-shared-deps asset size
async chunks size
distributable file count
page load bundle size
History
To update your PR or re-run it, just comment with: |
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.
Security changes LGTM! 👍 Thanks for addressing @cnasikas's comments @thompsongl, appreciate it! 🙂
* eui to v30.0.0 * removed unused RecentLinks component * update compressed -> display prop * euiformrow compressed -> display * euipopover remove withtitle * euibuttongroup prop updates * remove euibuttontoggle * src snapshot updates * tutorial euibuttongroup update * x-pack snapshot updates * euibuttongroup test updates * TODO * misc. js file updates * remove euinavdrawer scss references * translation removals * more x-pack snapshot updates * update data-test-subj lookup * icons logoAMP -> logoObservability * cypress selector updates * update euibuttontoggle onchange, test interaction * Fix auto-update toggle in Visualize * some more change -> click test updates * idtoselectedmap change * Fix feature table button group * Remove empty SASS file * Fix feature table button group II * Using `compressed` to also dictate dual range row * Fix a few security labels * An ML fix * Apply suggestions from code review Co-authored-by: Caroline Horn <[email protected]> * remove unused translation * update view_type_toggle snapshot * more snapshot updates * eui to 30.1.0 * undo temp ts-ignore * more x-pack snapshot updates * eui to 30.1.1 * update feature privilege text * Update x-pack/plugins/security/public/management/roles/edit_role/privileges/kibana/feature_table/sub_feature_form.tsx Co-authored-by: Joe Portner <[email protected]> * reuse i18n token * Adapt log entry action button to new `minWidth` prop * clean up label * Combine auto apply strings into one * ternary fix * cases status useCallback Co-authored-by: cchaos <[email protected]> Co-authored-by: Caroline Horn <[email protected]> Co-authored-by: Joe Portner <[email protected]> Co-authored-by: Felix Stürmer <[email protected]>
* eui to v30.0.0 * removed unused RecentLinks component * update compressed -> display prop * euiformrow compressed -> display * euipopover remove withtitle * euibuttongroup prop updates * remove euibuttontoggle * src snapshot updates * tutorial euibuttongroup update * x-pack snapshot updates * euibuttongroup test updates * TODO * misc. js file updates * remove euinavdrawer scss references * translation removals * more x-pack snapshot updates * update data-test-subj lookup * icons logoAMP -> logoObservability * cypress selector updates * update euibuttontoggle onchange, test interaction * Fix auto-update toggle in Visualize * some more change -> click test updates * idtoselectedmap change * Fix feature table button group * Remove empty SASS file * Fix feature table button group II * Using `compressed` to also dictate dual range row * Fix a few security labels * An ML fix * Apply suggestions from code review Co-authored-by: Caroline Horn <[email protected]> * remove unused translation * update view_type_toggle snapshot * more snapshot updates * eui to 30.1.0 * undo temp ts-ignore * more x-pack snapshot updates * eui to 30.1.1 * update feature privilege text * Update x-pack/plugins/security/public/management/roles/edit_role/privileges/kibana/feature_table/sub_feature_form.tsx Co-authored-by: Joe Portner <[email protected]> * reuse i18n token * Adapt log entry action button to new `minWidth` prop * clean up label * Combine auto apply strings into one * ternary fix * cases status useCallback Co-authored-by: cchaos <[email protected]> Co-authored-by: Caroline Horn <[email protected]> Co-authored-by: Joe Portner <[email protected]> Co-authored-by: Felix Stürmer <[email protected]> Co-authored-by: cchaos <[email protected]> Co-authored-by: Caroline Horn <[email protected]> Co-authored-by: Joe Portner <[email protected]> Co-authored-by: Felix Stürmer <[email protected]>
@thompsongl Thank you for addressing my comments! I appreciate it too! |
* master: Add derivative function (elastic#81178) [Discover] Deangularize context_app.html, part 3 (elastic#81838) [Visualize] Vis listing page breaks on unknown vis type (elastic#82018) Rename `batchSize` parameter to `batch_size` to be consisten with the API namings guidelines. (elastic#82123) Minor edits in Single Metric Viewer (elastic#82159) [Actions] Fix type contract (elastic#82168) Upgrade EUI to v30.1.1 (elastic#81499) Skip failing ES snapshot test (elastic#82207) Skip ES snapshot failing suite (elastic#82206) [Alerting UI] Grouped list of alert types using producers in Types filter of Alerts tab (elastic#81876) [Maps] convert vector style component to typescript round 1 (elastic#81961) Fix link to upgrade assistant (elastic#82138) Rename "service overview" to "service inventory" (elastic#81933) adjust policy test to drop test for server addresses (elastic#82120) Cleanup/codeowners (elastic#82146) [DOCS] Updates add data content (elastic#81093) [DOCS] Remove index mgmt docs (elastic#82099) [Search] fix cancelation related memory leaks (elastic#81996)
…e-details-overlay * 'master' of github.com:elastic/kibana: (72 commits) [CCR] Update README.md on how to start 2 clusters for testing (elastic#81487) [APM] Scale transaction rate correctly (elastic#82155) Upgrade to hapi version 18 (elastic#80468) [Uptime] Remove custom handling of license enabling (elastic#82019) [Telemetry] Remove `from` and `to` timestamps from usage stats APIs (elastic#81579) Enable send to background in Vega (elastic#82229) Enable send to background in Timelion (elastic#82232) [Actions & Connectors] removes Connector flyouts after usage (elastic#82126) Add derivative function (elastic#81178) [Discover] Deangularize context_app.html, part 3 (elastic#81838) [Visualize] Vis listing page breaks on unknown vis type (elastic#82018) Rename `batchSize` parameter to `batch_size` to be consisten with the API namings guidelines. (elastic#82123) Minor edits in Single Metric Viewer (elastic#82159) [Actions] Fix type contract (elastic#82168) Upgrade EUI to v30.1.1 (elastic#81499) Skip failing ES snapshot test (elastic#82207) Skip ES snapshot failing suite (elastic#82206) [Alerting UI] Grouped list of alert types using producers in Types filter of Alerts tab (elastic#81876) [Maps] convert vector style component to typescript round 1 (elastic#81961) Fix link to upgrade assistant (elastic#82138) ...
Summary
[email protected]
⏩[email protected]
Notable breaking changes:
All breaks have been accounted for. No action (beyond review) is required
EuiToggle
andEuiButtonToggle
. All current uses have been updated per "Upgrade path" guidelines in Fix EuiButtonGroup a11y and remove EuiToggle/EuiButtonToggle eui#4056.EuiNavDrawer
. The component had already been replaced withEuiCollapsibleNav
, and all residual code referencingEuiNavDrawer
assets has been removed.compressed
anddisplayOnly
props fromEuiFormRow
. All current uses have been updated to use the successordisplay
prop.EuiPopover
'swithTitle
prop. This prop has not provided any benefit for some time and there will be no visual difference with its removal.EuiButtonGroup
for a11y and introduced new required props for selection (idSelected
oridToSelectedMap
) and screen reader support (legend
).Note that
EuiKeyboardAccessible
has been deprecated and will be removed in a future major version.30.1.1
EuiInMemoryTable
types (#4179)Bug fixes
EuiButtonGroup
(#4186)EuiScreenReaderOnly
elements withinEuiButtonGroup
(#4189)30.1.0
EuiButtonGroup
focus, hover, selected and disabled states (#4142)display
prop toEuiToolTip
for common display block needs (#4148)EuiProgress
such asvis0
throughvis9
,warning
,success
and custom colors (#4130)affordForDisplacement
prop toEuiBottomBar
(#4156)width
property tomobileOptions
prop ofEuiTableRowCell
(#4169)Bug fixes
EuiComboBox
(#4162)EuiStepHorizontal
andEuiStepsHorizontal
(4166)30.0.0
EuiKeyboardAccessible
([4135]Deprecating EuiKeyboardAccessible eui#4135)EuiStep
,EuiHorizontalSteps
, andEuiCodeEditor
for better accessibility ([4135]Deprecating EuiKeyboardAccessible eui#4135)minWidth
prop toEuiButton
(4056)isSelected
prop to easily turnEuiButton
,EuiButtonEmpty
, andEuiButtonIcon
into toggle buttons (4056)EuiButtonGroup
props and render for better accessibility (4056)paddingSize
prop toEuiPopoverTitle
andEuiPopoverFooter
(4133)EuiBasicTable
types (#4125)EuiMarkdownEditor
plugin dependencies (4124)EuiMarkdownEditor
:react-dropzone
,rehype-*
,remark-*
, andunified
(#4124)Bug fixes
EuiSelectable
(4152)Breaking changes
EuiToggle
andEuiButtonToggle
in favor ofaria-pressed
(4056)legend
andidSelected
props ofEuiButtonGroup
to be required (4056)logoAPM
in favor oflogoObservability
(4065).euiHeaderNotification
CSS class (4065)EuiNavDrawer
in favor ofEuiCollapsibleNav
(4065)compressed
anddisplayOnly
props fromEuiFormRow
in favor of thedisplay
prop (4065)EuiPopover
'swithTitle
prop (4065)Theme: Amsterdam
line-height
for someEuiTitle
sizes (4133)EuiPopoverTitle
and fixed inherited padding fromEuiPopover
(4133)EuiFilterGroup
andEuiFilterButton
styles (4151)