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] Rearrange options #144891
[Lens] Rearrange options #144891
Conversation
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Async chunks
Unknown metric groupsAPI count
ESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM! tested it locally and works as expected. Thanx Joe :)
* main: [Lens] Rearrange options (elastic#144891) [Actionable Observability] Integrate alert search bar on rule details page (elastic#144718) [Security Solution] [Exceptions] Adds options to create a shared exception list and to create a single item from the manage exceptions view (elastic#144575) [Actionable Observability] Add context.alertDetailsUrl to connector template for Uptime > Monitor status & Uptime TLS rules (elastic#144740) [Security Solution] [Feat] Add Bulk Events to Timeline. (elastic#142737) [TIP] Env specific cypress config (elastic#144894) skip flaky suite (elastic#144885) [Enterprise Search] Fixes Search Index page to go blank when connection lost (elastic#144022) [Cloud Posture] track findings pages (elastic#144822) [ContentManagement] Inspector flyout (elastic#144240) [Cloud Posture] Dashboard Redesign - data counter cards (elastic#144565) [TIP] Run e2e pipeline on CI (elastic#144776) [Guided onboarding] Config updates for the Security guide (elastic#144844) Cleanup unused code for claiming tasks by id (elastic#144408) Ping the response-ops team whenever a new connector type is registered (elastic#144736)
Fixes #138075
Group by this field
This PR moves the "Group by this field" below the field selector. I renamed it to "Aggregate by this dimension first" (instead of field) because it's more accurate - this isn't really about the field and also a thing for the "Filters" function which doesn't have a field:
Drive-by change: Move the "Include empty rows" up above the granularity slider for intervals to match the date histogram order
Collapse by
It also moves "Collapse by" into the data section (relevant for xy breakdown, table rows and partition slices)
Also works for text based:
Technical details
For the collapse by part I had to add another form render hook to the visualization
renderDimensionEditorDataExtra
which is passed to the datasource dimension editor so it can be rendered in the right place. On the datasource side the integration for text based is very straight forward, for form based there are two flavors - just render below the operation edit options or pass to the operation editor (used for terms)