[Observability plugin] Audit new EUI Borealis theme

fkanout commented 1 month ago

Summary

It fixes #203338

Note

Some of the tokens have been ignored as they were on the server side or common folder, or they are coming from a shared package like packages/react/kibana_context

elasticmachine commented 1 month ago

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

fkanout commented 1 month ago

/oblt-deploy

elasticmachine commented 1 month ago

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

github-actions[bot] commented 1 month ago

:robot: GitHub comments

Expand to view the GitHub comments

Just comment with: - `/oblt-deploy` : Deploy a Kibana instance using the Observability test environments. - `run` `docs-build` : Re-trigger the docs validation. (use unformatted text in the comment!)

elasticmachine commented 1 month ago

:broken_heart: Build Failed

Buildkite Build
Commit: aac8ef418b16fd0ac1e62ea333b583b418f31a55
Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-204234-aac8ef418b16

Failed CI Steps

Test Failures

[job] [logs] FTR Configs #80 / Cloud Security Posture Findings Page onboarding Vulnerabilities - clicking on the No integrations installed prompt action button - install CNVM: navigates to the CNVM integration installation page
[job] [logs] Jest Tests #16 / EmptyBanner with one node does not render null
[job] [logs] Jest Tests #16 / EmptyBanner with one node does not render null
[job] [logs] Jest Tests #16 / Timeline TimelineAxisContainer should render with data
[job] [logs] Jest Tests #16 / Timeline TimelineAxisContainer should render with data
[job] [logs] Jest Tests #16 / Timeline VerticalLinesContainer should render with data
[job] [logs] Jest Tests #16 / Timeline VerticalLinesContainer should render with data
[job] [logs] Jest Tests #16 / useCytoscapeEventHandlers when data is received runs the layout
[job] [logs] Jest Tests #16 / useCytoscapeEventHandlers when data is received runs the layout
[job] [logs] Jest Tests #16 / useCytoscapeEventHandlers when data is received with a service name sets the primary class
[job] [logs] Jest Tests #16 / useCytoscapeEventHandlers when data is received with a service name sets the primary class

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
`apm`	1922	1929	+7

Async chunks

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

id	before	after	diff
`apm`	3.3MB	3.4MB	+9.6KB
`infra`	1.7MB	1.7MB	-78.0B
`synthetics`	1.1MB	1.1MB	+18.0B
total			+9.6KB

History

:broken_heart: Build #261305 failed d7be7b18eb7d90680c338f07a4281f96dee9b05d
:broken_heart: Build #260838 failed cd97121d456de58763380c633f995173abda55e3

cc @fkanout

fkanout commented 1 month ago

@kdelemme @mgadewoll This PR seems to overlap with other ones scope-wise. I discussed this with @jasonrhodes, and we agreed to scope this PR to update only the files that are owned by our team. Sorry about letting you double-review this work. But the new PR is significantly smaller. I appreciate your help!

patpscal commented 1 month ago

Hi @fkanout! To make our design review as helpful as possible, could you share from a final user's perspective where they might notice the changes (in case it impacts the user)?

fkanout commented 1 month ago

closing this in favor of https://github.com/elastic/kibana/pull/204615/

elastic / kibana