Skip to content

[APM]Changing status code colors on trace summary#47114

Merged
cauemarcondes merged 1 commit intoelastic:masterfrom
cauemarcondes:apm-90442
Oct 2, 2019
Merged

[APM]Changing status code colors on trace summary#47114
cauemarcondes merged 1 commit intoelastic:masterfrom
cauemarcondes:apm-90442

Conversation

@cauemarcondes
Copy link
Contributor

@cauemarcondes cauemarcondes commented Oct 2, 2019

closes #45406

Using light theme colors.

Before Dark/Light themes:
before-dark
before-light

After Dark/Light themes:
after-dark
after-light

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@sorenlouv sorenlouv left a comment

Choose a reason for hiding this comment

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

LGTM 🎨

@smith
Copy link
Contributor

smith commented Oct 21, 2019

Test plan: verified ✅

(there were more changes in the colors since this but these appear as expected.)

@smith smith added the apm:test-plan-done Pull request that was successfully tested during the test plan label Oct 21, 2019
@smith smith removed their assignment Oct 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apm:test-plan-done Pull request that was successfully tested during the test plan release_note:enhancement v7.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change status code colors on trace summary

4 participants