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

display NOT when new filter is negated #15865

Merged
merged 1 commit into from
Jan 9, 2018

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Jan 5, 2018

PR #15525 adds the ability to display an Others bucket for terms aggregations. When a user creates a filter on Others, it is just a negate filter on the Top N terms.

The Apply these filters UI does not provide any visual indication that a filter is negated and is a little confusing. This PR just adds NOT in front of a negated filter so users are presented with a better view of what the filter actually does.

screen shot 2018-01-05 at 8 54 02 am

Copy link
Contributor

@Bargs Bargs left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ppisljar ppisljar left a comment

Choose a reason for hiding this comment

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

LGTM

@nreese nreese merged commit 4f15c6e into elastic:master Jan 9, 2018
nreese added a commit to nreese/kibana that referenced this pull request Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants