Skip to content

[8.x] [ML] Replace legacy SCSS overwrites (#216698)#217348

Merged
kibanamachine merged 1 commit into
elastic:8.xfrom
kibanamachine:backport/8.x/pr-216698
Apr 7, 2025
Merged

[8.x] [ML] Replace legacy SCSS overwrites (#216698)#217348
kibanamachine merged 1 commit into
elastic:8.xfrom
kibanamachine:backport/8.x/pr-216698

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

Part of elastic#140695

This PR replaces all remaining SCSS overrides in ML.

> ml/public/application/_index.scss
> ./job_selector/_index.scss
> ./job_selector/_job_selector.scss
> ./rule_editor/_index.scss
> ./rule_editor/_rule_editor.scss
> ./anomalies_table/_index.scss
> ./anomalies_table/_anomalies_table.scss
>
data_visualizer/public/application/common/components/stats_table/components/field_data_row/column_chart.scss
>
data_visualizer/public/application/common/components/field_type_icon/_index.scss
>
data_visualizer/public/application/common/components/field_type_icon/_field_type_icon.scss
>
x-pack/platform/packages/private/ml/aiops_components/src/dual_brush/dual_brush.scss

There are minor color changes in the `dual brush` and `GanttBar` in the
Job Selector.

| Before  | After |
| ------------- | ------------- |
| <img width="1217" alt="dual-brush-before-light"
src="https://github.com/user-attachments/assets/e87f1600-c1f1-42ef-a4f8-a8d5b21e2ca7"
/> | <img width="881" alt="dual-brush-after-light"
src="https://github.com/user-attachments/assets/17996e65-50dc-42e8-9b9a-4757f2b30309"
/> |
| <img width="863" alt="dual-brush-before-dark"
src="https://github.com/user-attachments/assets/685fe511-b715-457b-8173-ece0d41f7bef"
/> | <img width="865" alt="dual-brush-after-dark"
src="https://github.com/user-attachments/assets/593b6085-281b-49b2-b0fd-9ae6f44b6684"
/> |
| <img width="576" alt="job_selector_before_light"
src="https://github.com/user-attachments/assets/ce2438c1-a54f-4066-bb4d-b86412d55e1e"
/> | <img width="564" alt="job_selector_after_light"
src="https://github.com/user-attachments/assets/74cb28d3-73c3-4836-ae53-f64f7730cf09"
/> |
| <img width="581" alt="job_selector_before_dark"
src="https://github.com/user-attachments/assets/7be93e06-12a0-4715-ac35-74711e08c761"
/> | <img width="567" alt="job_selector_after_dark"
src="https://github.com/user-attachments/assets/c3af92ee-f510-4f40-a99a-04f446652d91"
/> |
| <img width="574" alt="role_editor_before_light"
src="https://github.com/user-attachments/assets/8e8e33b8-2688-4526-9062-20dab205dcbf"
/> | <img width="564" alt="rule_editor_after_light"
src="https://github.com/user-attachments/assets/98142dfc-b74a-4bbd-af8c-c6c041805826"
/> |
| <img width="576" alt="role_editor_before_dark"
src="https://github.com/user-attachments/assets/e534c5f1-f75a-433d-91d8-dc57e059e407"
/> | <img width="572" alt="rule_editor_after_dark"
src="https://github.com/user-attachments/assets/2f56394f-4585-4176-a178-ef85394ab46d"
/> |
| <img width="572" alt="quick_role_editor_before_light"
src="https://github.com/user-attachments/assets/7f414295-e799-4073-84b8-d2bd94eb293f"
/> | <img width="567" alt="quick_role_editor_after_light"
src="https://github.com/user-attachments/assets/1d7bbc7b-bc44-4753-b9b6-6cac8cfb8953"
/> |
| <img width="578" alt="quick_role_editor_before_dark"
src="https://github.com/user-attachments/assets/f1d5291e-ef72-4e40-a614-b909193ec060"
/> | <img width="562" alt="quick_role_editor_after_dark"
src="https://github.com/user-attachments/assets/90c2927a-0bf2-4f70-a13c-7937ae2bf476"
/> |
| <img width="1205" alt="discover_vis_before_light"
src="https://github.com/user-attachments/assets/899311e7-d10b-48fe-91e1-95c3af7f5608"
/> | <img width="1201" alt="discover-vis-after-light"
src="https://github.com/user-attachments/assets/05f0dcde-6a1b-4139-95ba-19a24ad4fdcf"
/> |
| <img width="1207" alt="discover-vis-before-dark"
src="https://github.com/user-attachments/assets/cde9e49e-b9c3-4bd5-9bd5-32b4f09ce834"
/> | <img width="1207" alt="discover-viz-after-dark"
src="https://github.com/user-attachments/assets/544e44a9-3676-448d-9348-d88a67284a59"
/> |
| <img width="1160" alt="anomalies_table_before_light"
src="https://github.com/user-attachments/assets/9be79294-9808-4509-a1cb-02e342d9abe2"
/> | <img width="1136" alt="anomalies_table_after_light"
src="https://github.com/user-attachments/assets/3b3ce7ad-6f67-4caf-b12d-1839bb2c08ab"
/> |
| <img width="1152" alt="anomalies_table_before_dark"
src="https://github.com/user-attachments/assets/8e5af9fd-90f5-4f85-bd5b-40dc0ab74d0a"
/> | <img width="1144" alt="anomalies_table_after_dark"
src="https://github.com/user-attachments/assets/c6ca08b6-e816-49d2-8c15-9ec9bb1dd983"
/> |
| <img width="668" alt="category_examples_before"
src="https://github.com/user-attachments/assets/9bfd1978-27fe-41bc-9828-f94314e420a6"
/> | <img width="1101" alt="category_examples_after"
src="https://github.com/user-attachments/assets/72c58a22-6d1e-4901-898c-9c54c46eb3a9"
/> |

(cherry picked from commit 9f932a0)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 7, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) April 7, 2025 14:29
@kibanamachine kibanamachine merged commit f143d91 into elastic:8.x Apr 7, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
aiops 606 601 -5
dataVisualizer 757 743 -14
ml 2430 2421 -9
total -28

Async chunks

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

id before after diff
aiops 451.8KB 450.7KB -1.1KB
dataVisualizer 619.4KB 616.8KB -2.7KB
ml 5.4MB 5.4MB -9.3KB
total -13.1KB

cc @rbrtj

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants