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

[8.10] [ML] Fixes dark mode in flyouts and modals (#164399) #164559

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.10:

Questions ?

Please refer to the Backport tool documentation

A recent EUI change has caused a problem with the theme when using the
deprecated `toMountPoint` inside `overlays.openFlyout` to create
flyouts.
This causes the contents of the rendered flyout to not know the current
theme, this is obvious when running in dark mode.

The fix is to switch to the non-deprecated version of `toMountPoint`.

Flyouts:
Create anomaly detection job from Lens flyout in Dashboard.
Anomaly swim lane and anomaly chart job embeddables job selection flyout
in Dashboard.
Log pattern analysis flyout in Discover.

Modals:
Trained models start deployment modal.
Trained models force stop deployment modal.
Trained models stop deployment modal when there are multiple
deployments.

Misc:
Page not found banner.
Jobs list header, which contains the settings button.
DFA clone job warning toast when the original data view no longer
exists.
Components in ml's date picker package

Fixes elastic#164379

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Dima Arnautov <[email protected]>
(cherry picked from commit af440aa)
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
dataVisualizer 540 554 +14
ml 1793 1807 +14
transform 365 379 +14
total +42

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/ml-date-picker 4 0 -4

Async chunks

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

id before after diff
aiops 595.3KB 593.5KB -1.8KB
dataVisualizer 605.0KB 606.3KB +1.3KB
infra 2.0MB 2.0MB +7.0B
ml 3.5MB 3.5MB +1.4KB
transform 404.7KB 406.2KB +1.4KB
total +2.3KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
dataVisualizer 23.1KB 23.1KB +56.0B
transform 17.8KB 17.8KB +56.0B
total +112.0B
Unknown metric groups

API count

id before after diff
@kbn/ml-date-picker 52 47 -5
aiops 60 61 +1
total -4

References to deprecated APIs

id before after diff
@kbn/ml-date-picker 1 0 -1
aiops 19 6 -13
dataVisualizer 31 25 -6
ml 176 156 -20
transform 64 62 -2
total -42

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

cc @jgowdyelastic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants