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] Change chart switcher design #89708

Closed
flash1293 opened this issue Jan 29, 2021 · 2 comments · Fixed by #91844
Closed

[Lens] Change chart switcher design #89708

flash1293 opened this issue Jan 29, 2021 · 2 comments · Fixed by #91844
Assignees
Labels
enhancement New value added to drive a business result Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@flash1293
Copy link
Contributor

To be able to handle more chart types the chart switcher should be re-designed as shown in the screenshots below. The current chart has a checkmark icon, incompatible charts have a warning triangle. On hover the warning triangle should show an explanation what would happen on switch.

Screenshot 2021-01-28 at 10 42 11

Screenshot 2021-01-29 at 13 29 34

@chandlerprall Is it possible to build this today using the standard list group component?

@flash1293 flash1293 added Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Lens reason:enhancement labels Jan 29, 2021
@elasticmachine
Copy link
Contributor

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

@chandlerprall
Copy link
Contributor

Is it possible to build this today using the standard list group component?

I think EuiSelectable is a better fit https://elastic.github.io/eui/#/forms/selectable

@dej611 dej611 self-assigned this Feb 15, 2021
@sophiec20 sophiec20 added the enhancement New value added to drive a business result label Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants