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

Bumping EUI to v70.4.0 #145888

Merged
merged 21 commits into from Dec 2, 2022
Merged

Bumping EUI to v70.4.0 #145888

merged 21 commits into from Dec 2, 2022

Conversation

1Copenut
Copy link
Contributor

@1Copenut 1Copenut commented Nov 21, 2022

eui@70.2.4eui@70.4.0

  • "Fixed EuiButtonGroup firing onChange twice" required changing some tests from click to change

70.4.0

  • Updated EuiTourStep.footerAction type to accept ReactNode[] (#6384)
  • Vertically aligned all footer content so that euiTourStepIndicator is always centered (#6384)
  • Added filterInCircle glyph to EuiIcon (#6385)
  • Added color prop to EuiBeacon (#6420)
  • Added the euiMaxBreakpoint and euiMinBreakpoint CSS-in-JS utilities for creating min/max-width media queries (#6431)

Bug fixes

  • Restores the previous match operator behaviour when the query value is split into multiple terms after analysis. (#6409)
  • Fixed missing slide-in animation on EuiCollapsibleNavs and left-side EuiFlyouts (#6422)
  • Fix bug in EuiCard where footer were not aligned to the bottom of the card (#6424)
  • Fixed multiple component media queries for consumers with custom theme breakpoints (#6431)

70.3.0

  • EuiSearchBar now automatically wraps special characters not used by query syntax in quotes (#6356)
  • Added alignment prop to EuiBetaBadge (#6361)
  • EuiButton now accepts minWidth={false} (#6373)

Bug fixes

  • Fixed EuiPageTemplate not correctly passing the component prop to the inner main content wrapper. (#6352)
  • EuiSkipLink now correctly calls onClick even when fallbackDestination is invalid (#6355)
  • Permanently fixed EuiModal to not cause scroll-jumping issues on modal open (#6360)
  • Re-fixed EuiPageSection not correctly merging contentProps.css (#6365)
  • Fixed EuiTab not defaulting to size m (#6366)
  • Fixed the shadow sizes of .eui-yScrollWithShadows and .eui-xScrollWithShadows (#6374)
  • Fixed bug in EuiCard where the inner content in vertical cards was not growing 100% in width (#6377)
  • Fixed incorrect margins in EuiSuperDatePicker caused by EuiFlex CSS gap change (#6380)
  • Fixed visual bug in nested EuiFlexGroups, where the parent EuiFlexGroup is responsive but a child EuiFlexGroup is not (#6381)

CSS-in-JS conversions

  • Converted EuiModal to Emotion (#6321)

Fixes

  • EuiButton no longer outputs unnecessary inline styles for minWidth={0} or minWidth={false} (#6373)
  • EuiFacetButton no longer reports type issues when passing props accepted by EuiButton (#6373)

@1Copenut 1Copenut added release_note:skip Skip the PR/issue when compiling release notes EUI ci:cloud-deploy Create or update a Cloud deployment v8.7.0 labels Nov 21, 2022
@1Copenut 1Copenut self-assigned this Nov 21, 2022
@1Copenut 1Copenut changed the title [DO NOT MERGE] Bumping EUI to v70.3.0 [DO NOT MERGE] Bumping EUI to v70.4.0 Nov 23, 2022
@1Copenut 1Copenut marked this pull request as ready for review November 23, 2022 22:01
@1Copenut 1Copenut requested review from a team as code owners November 23, 2022 22:01
Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

Core changes LGTM

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.

ResponseOps changes LGTM

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.

Left one question about a test change for Deployment Management

Copy link
Contributor

@michaelolo24 michaelolo24 left a comment

Choose a reason for hiding this comment

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

Threat hunting investigations changes. LGTM!

package.json Outdated Show resolved Hide resolved
Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

Global Exp changes LGTM

Copy link
Member

@cee-chen cee-chen left a comment

Choose a reason for hiding this comment

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

🎉 Woohoo! Nice to have a smaller upgrade for a change.

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.

Deployment Management changes LGTM

Copy link
Contributor

@paul-tavares paul-tavares left a comment

Choose a reason for hiding this comment

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

LGTM - on behalf of @elastic/security-onboarding-and-lifecycle-mgt

Copy link
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

LGTM!

@kibana-ci
Copy link
Collaborator

kibana-ci commented Dec 1, 2022

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
infra 1.1MB 1.1MB +912.0B
licenseManagement 63.4KB 63.3KB -20.0B
security 549.5KB 549.6KB +124.0B
visTypeVega 1.8MB 1.8MB +912.0B
total +1.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 343.8KB 340.5KB -3.4KB
kbnUiSharedDeps-npmDll 5.2MB 5.2MB +7.1KB
total +3.7KB
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 109 115 +6
securitySolution 442 448 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 110 117 +7
securitySolution 519 525 +6
total +21

History

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

cc @1Copenut

@1Copenut 1Copenut merged commit 7376d9c into elastic:main Dec 2, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Dec 2, 2022
@1Copenut 1Copenut deleted the feature/eui-70.3.0 branch December 2, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting ci:cloud-deploy Create or update a Cloud deployment EUI release_note:skip Skip the PR/issue when compiling release notes v8.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet