Conversation
…workspace_panel_wrapper.scss Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>
…into lens/fitting-functions
mbondyra
left a comment
There was a problem hiding this comment.
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? |
| hasDividers | ||
| /> | ||
| </EuiFormRow> | ||
| <EuiHorizontalRule /> |
There was a problem hiding this comment.
This needs to be outside of the tooltip and form row.
| <EuiHorizontalRule /> | |
| <EuiHorizontalRule size="s" /> |
There was a problem hiding this comment.
You mean margin="s"?
|
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. |
|
@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 OutStack TraceBuild 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_legendConfigis extended with a new flagshowSingleSeriesto 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)