Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[APM] Update beta color and remove search button #21643

Merged
merged 1 commit into from
Aug 3, 2018

Conversation

sorenlouv
Copy link
Member

@sorenlouv sorenlouv commented Aug 3, 2018

screen shot 2018-08-03 at 14 24 50

@sorenlouv sorenlouv added the Team:APM All issues that need APM UI Team support label Aug 3, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@elasticmachine
Copy link
Contributor

💔 Build Failed

@sorenlouv
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@sorenlouv sorenlouv merged commit 23d8d6a into elastic:master Aug 3, 2018
@sorenlouv sorenlouv deleted the remove-search-button branch August 3, 2018 15:40
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Aug 3, 2018
# Conflicts:
#	x-pack/plugins/apm/public/components/shared/KueryBar/Typeahead/index.js
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Aug 3, 2018
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Aug 3, 2018
sorenlouv added a commit that referenced this pull request Aug 4, 2018
…h button (#21643) (#21672)

* [APM] Kuery fixes (#21272)

* Links should have basepath

* Fix empty state messages

* Fix padding

* Fix beta label

* Change copy of beta label

* Small font size

* Update beta color and remove search button (#21643)
@formgeist
Copy link
Contributor

@sqren @makwarth Just out of curiosity; why was the tooltip and search button removed?

@makwarth
Copy link

makwarth commented Aug 6, 2018

@formgeist See #20924

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:APM All issues that need APM UI Team support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants