Skip to content

[Discover] SCSS to Emotions migration part 3/4#222947

Merged
akowalska622 merged 14 commits intoelastic:mainfrom
akowalska622:discover-scss-migration-part-3
Jun 16, 2025
Merged

[Discover] SCSS to Emotions migration part 3/4#222947
akowalska622 merged 14 commits intoelastic:mainfrom
akowalska622:discover-scss-migration-part-3

Conversation

@akowalska622
Copy link
Contributor

@akowalska622 akowalska622 commented Jun 6, 2025

Summary

Parent issue: [DataDiscovery] Replace SCSS with CSS-in-JS

Followed Emotion standards guide

Part 3 of SCSS -> Emotion migration. Files included:

Part 3

  • src/platform/plugins/shared/field_formats/public/index.scss
  • src/platform/plugins/shared/field_formats/public/lib/content_types/_html_content_type.scss
  • src/platform/plugins/shared/field_formats/public/lib/content_types/_index.scss
  • src/platform/plugins/shared/field_formats/public/lib/converters/_index.scss
  • src/platform/plugins/shared/field_formats/public/lib/converters/_string.scss
  • src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_source/source.scss
  • src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_table/table.scss
  • src/platform/plugins/shared/unified_doc_viewer/public/components/json_code_editor/json_code_editor.scss

@akowalska622 akowalska622 changed the title WIP html_content_type styles refactor [Discover] SCSS to Emotions migration part 3/4 Jun 6, 2025
@akowalska622 akowalska622 self-assigned this Jun 10, 2025
@akowalska622 akowalska622 marked this pull request as ready for review June 10, 2025 13:16
@akowalska622 akowalska622 requested review from a team as code owners June 10, 2025 13:16
@akowalska622 akowalska622 added Feature:Discover Discover Application technical debt Improvement of the software architecture and operational architecture release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// labels Jun 10, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@akowalska622 akowalska622 added backport:version Backport to applied version labels v8.19.0 labels Jun 10, 2025
Copy link
Contributor

@l-suarez l-suarez left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
fieldFormats 70 57 -13
unifiedDocViewer 342 327 -15
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
alerting 91.4KB 91.6KB +159.0B
cases 1.3MB 1.3MB +159.0B
discover 1.1MB 1.1MB +159.0B
ml 5.4MB 5.4MB +159.0B
observability 1.3MB 1.3MB +159.0B
securitySolution 9.3MB 9.3MB +159.0B
transform 622.8KB 622.9KB +159.0B
unifiedDocViewer 231.6KB 223.0KB -8.7KB
total -7.6KB

Page load bundle

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

id before after diff
core 456.7KB 456.8KB +149.0B
esUiShared 88.4KB 88.4KB -4.0B
fieldFormats 62.1KB 57.4KB -4.7KB
fleet 167.4KB 167.6KB +159.0B
unifiedDocViewer 12.6KB 12.7KB +132.0B
total -4.3KB

History

cc @akowalska622

@akowalska622 akowalska622 merged commit adffdde into elastic:main Jun 16, 2025
10 checks passed
@akowalska622 akowalska622 deleted the discover-scss-migration-part-3 branch June 16, 2025 15:04
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/15684515150

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 222947

Questions ?

Please refer to the Backport tool documentation

@akowalska622
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 17, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @akowalska622

akowalska622 added a commit that referenced this pull request Jun 18, 2025
)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Discover] SCSS to Emotions migration part 3/4
(#222947)](#222947)

<!--- Backport version: 10.0.1 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Ania
Kowalska","email":"63072419+akowalska622@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-06-16T15:04:47Z","message":"[Discover]
SCSS to Emotions migration part 3/4 (#222947)\n\n## Summary\n\nParent
issue: [[DataDiscovery] Replace SCSS
with\nCSS-in-JS](https://github.com/elastic/kibana/issues/209807#top)\n\nFollowed
[Emotion
standards\nguide](https://docs.google.com/document/d/1CPflY8yCc3lZDg2BQkaMTgIkZlqiAEyMcAAvZbsjcTc/edit?pli=1&tab=t.0#heading=h.4zj1jq66y5an)\n\nPart
3 of SCSS -> Emotion migration. Files included:\n\n### Part 3\n-
`src/platform/plugins/shared/field_formats/public/index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_html_content_type.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_string.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_source/source.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_table/table.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/json_code_editor/json_code_editor.scss`","sha":"adffdde56d7af029432930638df4b9d2ee523d1f","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Discover","technical
debt","release_note:skip","Team:DataDiscovery","backport:version","v9.1.0","v8.19.0"],"title":"[Discover]
SCSS to Emotions migration part
3/4","number":222947,"url":"https://github.com/elastic/kibana/pull/222947","mergeCommit":{"message":"[Discover]
SCSS to Emotions migration part 3/4 (#222947)\n\n## Summary\n\nParent
issue: [[DataDiscovery] Replace SCSS
with\nCSS-in-JS](https://github.com/elastic/kibana/issues/209807#top)\n\nFollowed
[Emotion
standards\nguide](https://docs.google.com/document/d/1CPflY8yCc3lZDg2BQkaMTgIkZlqiAEyMcAAvZbsjcTc/edit?pli=1&tab=t.0#heading=h.4zj1jq66y5an)\n\nPart
3 of SCSS -> Emotion migration. Files included:\n\n### Part 3\n-
`src/platform/plugins/shared/field_formats/public/index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_html_content_type.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_string.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_source/source.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_table/table.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/json_code_editor/json_code_editor.scss`","sha":"adffdde56d7af029432930638df4b9d2ee523d1f"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/222947","number":222947,"mergeCommit":{"message":"[Discover]
SCSS to Emotions migration part 3/4 (#222947)\n\n## Summary\n\nParent
issue: [[DataDiscovery] Replace SCSS
with\nCSS-in-JS](https://github.com/elastic/kibana/issues/209807#top)\n\nFollowed
[Emotion
standards\nguide](https://docs.google.com/document/d/1CPflY8yCc3lZDg2BQkaMTgIkZlqiAEyMcAAvZbsjcTc/edit?pli=1&tab=t.0#heading=h.4zj1jq66y5an)\n\nPart
3 of SCSS -> Emotion migration. Files included:\n\n### Part 3\n-
`src/platform/plugins/shared/field_formats/public/index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_html_content_type.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/content_types/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_index.scss`\n-\n`src/platform/plugins/shared/field_formats/public/lib/converters/_string.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_source/source.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/doc_viewer_table/table.scss`\n-\n`src/platform/plugins/shared/unified_doc_viewer/public/components/json_code_editor/json_code_editor.scss`","sha":"adffdde56d7af029432930638df4b9d2ee523d1f"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes scss-removal Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// technical debt Improvement of the software architecture and operational architecture v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants