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

[Lens] Update grouping editor to indicate the expected result of a grouping change #76904

Merged
merged 1 commit into from
Sep 8, 2020

Conversation

mbondyra
Copy link
Contributor

@mbondyra mbondyra commented Sep 8, 2020

Fixes #76038

Summary

The copy we have currently for date histogram feels unclear. Grouping editor for date histograms should reflect the changes in the other operation, not in the date order.

Before:

After:

Checklist

Delete any items that are not applicable to this PR.

@mbondyra mbondyra added Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Lens v7.10.0 labels Sep 8, 2020
@mbondyra mbondyra requested review from wylieconlon, dej611, flash1293 and a team September 8, 2020 08:26
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@mbondyra mbondyra force-pushed the lens/update_bucket_editor_copy branch from babd34f to e708792 Compare September 8, 2020 08:46
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
lens 874.0KB +36.0B 873.9KB

History

  • 💔 Build #73016 failed babd34f5c16e139b557ac5f75d31ddab4def806a

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

Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Tested in Chrome, LGTM

@mbondyra mbondyra merged commit 728dfb4 into elastic:master Sep 8, 2020
@mbondyra mbondyra deleted the lens/update_bucket_editor_copy branch September 8, 2020 15:40
mbondyra added a commit to mbondyra/kibana that referenced this pull request Sep 8, 2020
mbondyra added a commit that referenced this pull request Sep 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Lens] Update grouping editor to indicate the expected result of a grouping change
4 participants