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 11.3.2 #38262

Merged
merged 28 commits into from
Jun 14, 2019
Merged

Upgrade EUI to 11.3.2 #38262

merged 28 commits into from
Jun 14, 2019

Conversation

thompsongl
Copy link
Contributor

Summary

11.0.111.3.1

Notable: EuiColorPicker has been rebuilt and restyled, but with a compatible API.

11.3.1

Bug fixes

  • Fixed EuiBadge conflicts with providing both iconOnClick and onClick (#1994)
  • Fixed optional TS definitions for EuiColorPicker onBlur and onFocus callbacks (#1993)
  • Fixed EuiIcon again so that webpack can build dynamic require contexts (#1998)
  • Fixed double borders on prepend/append items in EuiFormControlLayout (#1996)
  • Fixed EuiSuperSelect TS definitions (#1995)

11.3.0

  • Converted EuiTableRowHeaderCheckbox to TS (#1973)
  • Added missing TypeScript definition for EuiFieldText's compressed prop (#1977)
  • Converted EuiTableRowCellCheckbox to TS (#1964)
  • Updated caniuse-lite version resolution (#1970)
  • Added a webpack directive for naming icon chunks (#1944)
  • Added ability to update EuiInMemoryTable sorting prop and remove columns after sorting is applied (#1972)
  • Added onToggle callback to EuiAccordion (#1974)
  • Removed options defaultProps value from EuiSuperSelect (#1975)
  • Removed TSlint and will perform all linting through ESLint (#1950)
  • Added new component EuiDelayRender (#1876)
  • Replaced EuiColorPicker with custom, customizable component (#1914)
  • Added jsx-a11y eslint plugin and rules to match Kibana (#1952)
  • Changed EuiCopy beforeMessage prop to accept node instead of just string (#1952)

Bug fixes

  • Fixed environment setup for running test-unit script on Windows (#1971)
  • Fixed focus on single selection of EuiComboBox (#1965)
  • Fixed type mismatch between PropType and TypeScript def for EuiGlobalToastList toast title (#1978)
  • Fixed missing Typescript definition for EuiButton's color="text" option (#1980)
  • Fixed Prettier formatting lint error in EuiTable TS def file (#1986)
  • Fixed not clickable button with svg in Safari (#1985)
  • Fixed EuiToggle pointer events for those using icons only (#1991)

11.2.1

Bug fixes

  • Fixed type mismatch between PropType and TypeScript def for EuiToast title (#1962)

11.2.0

  • Converted EuiFormControlLayoutCustomIcon to TS (#1956)
  • Converted EuiStepNumber to TS (#1893)
  • Converted EuiFormControlLayoutClearButton to TS (#1922)
  • Added data-test-subj property to EuiDraggable and EuiDroppable (#1943)
  • Added type definitions to EuiSuperSelect (#1907)
  • Updated EuiIcon to use Slack's updated branding (#1954)
  • Updated compile-icons script to format icon components with Prettier (#1955)

Bug fixes

  • Addressed a chrome issue where negative letter-spacing can reverse RTL text in SVGs (#1960)

11.1.0

  • Converted pretty_interval to TS (#1920)
  • Converted relative_options to TS (#1921)
  • Added width to EuiFlexItem when gutter in EuiFlexGrid is set to none. (#1941)
  • Format all JavaScript files with Prettier through ESLint (#1906)
  • Replaced appSecurityAnalytics in EuiIcon with an updated SVG (#1948)

Bug fixes

  • Removed unused prop enum of l in EuiButton (#1936)
  • Fixed EuiSelect browser event inconsistencies by normalizing mouseup propagation (#1926)
  • Removed children as a required prop for EuiOverlayMask (#1937)

Copy link
Contributor

@chandlerprall chandlerprall left a comment

Choose a reason for hiding this comment

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

one devex change requested, otherwise looks good!

@thompsongl thompsongl requested a review from a team June 6, 2019 15:37
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
Contributor

@cqliu1 cqliu1 left a comment

Choose a reason for hiding this comment

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

No issues with the updates to the Canvas snapshots. Only EUI component related code was updated in the snapshots. I just had one question about the types in the FontPicker component.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
Contributor

@PhilippBaranovskiy PhilippBaranovskiy left a comment

Choose a reason for hiding this comment

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

In the existing issues I know that code looks good.

@spalger
Copy link
Contributor

spalger commented Jun 12, 2019

Seems we were duplicating work in #38434, let's use this PR instead

@thompsongl
Copy link
Contributor Author

Seems we were duplicating work in #38434, let's use this PR instead

We found a bug introduced in the jump, too, so this will go to 11.3.2 once EUI is updated

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@thompsongl thompsongl changed the title Upgrade EUI to 11.3.1 Upgrade EUI to 11.3.2 Jun 14, 2019
@thompsongl thompsongl added the release_note:skip Skip the PR/issue when compiling release notes label Jun 14, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@legrego legrego left a comment

Choose a reason for hiding this comment

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

Since the new color picker allows for invalid hex, it's possible for users to get themselves into a state where their space won't save:

image

The components on this page really aren't setup to quickly fix this, and I don't want to block your PR on this edge case. I'll open an issue for us to correct it once this merges.

Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Ran local. Issues look fixed.

@thompsongl thompsongl merged commit c22b706 into elastic:master Jun 14, 2019
chandlerprall pushed a commit that referenced this pull request Jun 17, 2019
* Upgrade EUI to 11.3.2 (#38262)

* eui 11.3.1

* EuiSuperSelect TS updates

* EuiColorPicker updates

* EuiBadge updates

* a11y updates

* i18n service updates

* remove unused translations

* move valueOfSelected logic to private method

* update EuiColorPicker class name

* Updated canvas storyshots

* i18n fixes; maps color selection position

* i18n snapshot updates

* update canvas storyshots

* snapshot updates

* 11.3.2

* maps colorpicker

* space avatar colorpicker

* update i18n json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore release_note:skip Skip the PR/issue when compiling release notes v7.3.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants