-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Legend config #70619
[Lens] Legend config #70619
Conversation
…workspace_panel_wrapper.scss Co-authored-by: Caroline Horn <[email protected]>
…into lens/fitting-functions
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.
I will continue with testing the PR tomorrow. When it comes to the code, I've left two small notes.
x-pack/plugins/lens/public/pie_visualization/render_function.tsx
Outdated
Show resolved
Hide resolved
@elasticmachine merge upstream |
Tested on FF, works fine. I've resolved the conflicts with master, I think it's fine, but @flash1293 feel free to check if I didn't mess up anything. |
x-pack/plugins/lens/public/xy_visualization/xy_config_panel.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/lens/public/xy_visualization/xy_config_panel.tsx
Outdated
Show resolved
Hide resolved
Thanks for the comments and special thanks for the master merge - I think I addressed all comments, @cchaos can you take another look? |
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.
hasDividers | ||
/> | ||
</EuiFormRow> | ||
<EuiHorizontalRule /> |
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.
This needs to be outside of the tooltip and form row.
<EuiHorizontalRule /> | |
<EuiHorizontalRule size="s" /> |
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.
You mean margin="s"
?
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.
Yep 😆
Welp, sorry for that @cchaos - I had to resolve a conflict before pushing which I thought was trivial. Of course it wasn't. Should be fixed now. |
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.
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / X-Pack Alerting API Integration Tests.x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/update·ts.alerting api integration security and spaces enabled Alerts update space_1_all at space1 should handle updates to an alert schedule by rescheduling the underlying taskStandard Out
Stack Trace
Build metricspage load bundle size
History
To update your PR or re-run it, just comment with: |
* master: (34 commits) Adds Role Based Access-Control to the Alerting & Action plugins based on Kibana Feature Controls (elastic#67157) [Monitoring] Revert direct shipping code (elastic#72505) Use server basepath when creating reporting jobs (elastic#72722) Adding api test for transaction_groups /breakdown and /avg_duration_by_browser (elastic#72623) [Task Manager] Addresses flaky test introduced by buffered store (elastic#72815) [Observability] filter "hasData" api by processor event (elastic#72810) do not pass title as part of tsvb request (elastic#72619) [Lens] Legend config (elastic#70619) Stabilize closing toast (elastic#72097) stabilize failing test (elastic#72086) Stabilize filter bar test (elastic#72032) Unskip vislib tests (elastic#71452) [ML] Fix layout of anomaly chart tooltip for long field values (elastic#72689) fix preAuth/preRouting mocks (elastic#72663) [Security Solution] Hide KQL bar (all pages) and alerts filters (Detections) when Resolver is full screen (elastic#72788) [Uptime] Rename Whitelist to Allowlist in parse_filter_map (elastic#71584) [Security Solution] Fixes exception modal not loading content (elastic#72770) [Security Solution][Exceptions] - Require non empty entries and non empty string values in exception list items (elastic#72748) [Detections] Add validation for Threshold value field (elastic#72611) [SIEM][Detection Engine][Lists] Adds version and immutability data structures (elastic#72730) ...
Fixes #67310
Fixes #68293
Fixes #71500
This PR completes the legend options in pie and xy charts:
XY chart:
Pie chart:
It also moves the pie settings from the layer widget into the toolbar to be consistent with xy chart:
TODOs:
Expression changes
lens_xy_legendConfig
is extended with a new flagshowSingleSeries
to encode the current "auto" mode in a BWC fashion. Everything else is still available on the expression layer and below.The pie chart render function takes another argument
legendPosition
(everything else is already available)