-
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 v99.2.0-borealis.0 #209690
base: main
Are you sure you want to change the base?
Conversation
4c1d4f8
to
980eee2
Compare
980eee2
to
7c62af4
Compare
@elasticmachine merge upstream |
Pinging @elastic/eui-team (EUI) |
💔 Build Failed
Failed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
Page load bundle
History
cc @tkajtoch |
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.
codeowner changes lgtm
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
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.
Kibana-Security changes LGTM. Just a nit for non-snapshot testing.
@@ -78,7 +78,7 @@ test('updates identifier, initials and color when name is changed', () => { | |||
id: 'space-2', | |||
name: 'Space 2', | |||
initials: 'S2', | |||
color: '#9170B8', | |||
color: '#BFDBFF', |
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.
Nit: it would be nice if these color values were constants somewhere that could be referenced across all tests.
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 changes for ML LGTM - gave the color changes for the NER trained model test a visual check.
{ key: '8,529.2 - 11,335.66', name: '8,529.2 - 11,335.66', color: '#c8deff' }, | ||
{ key: '11,335.66 - 14,142.1', name: '11,335.66 - 14,142.1', color: '#f6f9fc' }, | ||
{ key: '14,142.1 - 16,948.55', name: '14,142.1 - 16,948.55', color: '#ffccc6' }, | ||
{ key: '7,747 - 8,529.2', name: '7,747 - 8,529.2', color: '#61a2ff' }, |
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.
Not sure why this label would change here?
99.1.0-borealis.0
⏩99.2.0-borealis.0
Questions? Please see our Kibana upgrade FAQ.
v99.2.0
EuiFieldText
styles to prioritizedisabled
styling overreadonly
. (#8271)plugs
andweb
glyphs toEuiIcon
(#8285)title
onEuiColorPalettePickerPaletteProps
to be optional (#8289)Bug fixes
disabled
behavior ofEuiFieldText
to prevent input changes. (#8271)Borealis changes
As you can probably tell, there are plenty of updated tests and snapshots this time around. They are the result of the following:
@elastic/eui
v99.2.0 to Borealis, which led to kibana tests usingrenderWithTheme
,mountWithTheme
, and other utility functions that wrap the tested component in<EuiThemeProvider>
to switch to the correct, new color values. With this change, all tests should now use Borealis, regardless of the testing framework or its configuration.css-
wprskz
-euiButtonIcon-xs-empty-primary
) regenerated.Most Borealis-related changes in this PR are test env only and have no impact on what users see when using Kibana.