Skip to content

Update comparison series styles to match the main series#123858

Merged
gbamparop merged 5 commits intoelastic:mainfrom
gbamparop:update-comparison-series-style
Jan 28, 2022
Merged

Update comparison series styles to match the main series#123858
gbamparop merged 5 commits intoelastic:mainfrom
gbamparop:update-comparison-series-style

Conversation

@gbamparop
Copy link
Copy Markdown
Contributor

@gbamparop gbamparop commented Jan 26, 2022

Summary

Comparison series styles were updated to match the main series.

euiPaletteColorBlind()is now used for both current and previous (comparison) time series.

Screen.Recording.2022-01-27.at.11.06.23.mov

Closes #121882

@gbamparop gbamparop requested a review from a team January 27, 2022 12:05
@gbamparop gbamparop marked this pull request as ready for review January 27, 2022 12:05
@gbamparop gbamparop added auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes v8.1.0 Team:APM - DEPRECATED Use Team:obs-ux-infra_services. labels Jan 27, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/apm-ui (Team:apm)

@gbamparop
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

merge conflict between base and head

@gbamparop gbamparop force-pushed the update-comparison-series-style branch from 990a682 to fa7695a Compare January 27, 2022 12:16
Copy link
Copy Markdown
Contributor

@formgeist formgeist left a comment

Choose a reason for hiding this comment

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

Design LGTM - great work @gbamparop 👍

@gbamparop gbamparop requested a review from a team January 27, 2022 16:26
Copy link
Copy Markdown
Contributor

@kpatticha kpatticha left a comment

Choose a reason for hiding this comment

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

matching colors 🥳 looks much better now
image

@gbamparop
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1182 1183 +1

Async chunks

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

id before after diff
apm 2.8MB 2.8MB +3.0KB

History

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

@gbamparop gbamparop merged commit b8e9aa0 into elastic:main Jan 28, 2022
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Jan 28, 2022
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 Backport failed

The pull request could not be backported due to the following error:
There are no branches to backport to. Aborting.

How to fix

Re-run the backport manually:

node scripts/backport --pr 123858

Questions ?

Please refer to the Backport tool documentation

jloleysens added a commit to jloleysens/kibana that referenced this pull request Jan 28, 2022
…fix-potential-race-condition-when-screenshotting

* 'main' of github.com:elastic/kibana: (75 commits)
  [Reporting] Logging improvements while generating reports (elastic#123802)
  [Uptime] Default alert connectors email settings (elastic#123244)
  Update comparison series styles to match the main series (elastic#123858)
  [RAC][Uptime] remove extra dot from the uptime alert connector message (elastic#124000)
  [Exploratory view] Allow ability add extra actions in lens embeddable (elastic#123713)
  [SecuritySolution][Investigations] Add message about missing index in data view in analyzer (elastic#122859)
  [TSVB] Formatting in the left axis is not respected when I have two separate axis (elastic#123903)
  [Discover] Remove services from component dependencies (elastic#121691)
  Stop IM rule execution if there are no events (elastic#123811)
  [Security Solution][Endpoint] Update Fleet Trusted Apps and Host Isolation Exception cards to use exception list summary API (elastic#123900)
  [Security Solution][Exceptions] Switches modal to flyout component (elastic#123408)
  [Workplace Search] Fix bug where modal visible after deleting a group (elastic#123976)
  [Alerting] Remove state variables from action variable menu (elastic#123702)
  replace deprecated api usage (elastic#123970)
  Fix package policy merge logic for boolean values (elastic#123974)
  [Security Solution][Endpoint][Policy] Remove GET policy list api route (elastic#123873)
  Reenable alert_add test suite (elastic#123862)
  [Fleet] Remove usage of IFieldType in Fleet (elastic#123960)
  [Lists] Add an instance of `ExceptionListClient` with server extension points turned off to context object provided to callbacks (elastic#123885)
  [Maps] Add execution context (elastic#123651)
  ...

# Conflicts:
#	x-pack/plugins/screenshotting/server/browsers/chromium/driver_factory/index.ts
awahab07 pushed a commit to awahab07/kibana that referenced this pull request Jan 31, 2022
)

* Update comparison series styles to match the main series
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:APM - DEPRECATED Use Team:obs-ux-infra_services. v8.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[APM] Charts: Update comparison series styles to match the main series

6 participants