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] Enabling metric trend line should not enable layer deletion #143687

Closed
drewdaemon opened this issue Oct 19, 2022 · 2 comments · Fixed by #143781
Closed

[Lens] Enabling metric trend line should not enable layer deletion #143687

drewdaemon opened this issue Oct 19, 2022 · 2 comments · Fixed by #143781
Labels
bug Fixes for quality problems that affect the customer experience Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@drewdaemon
Copy link
Contributor

(Unreleased)

Metric trend lines works by adding a hidden layer. Our current layer deleting setup changes from clearing a layer to deleting a layer if there is more than one. In the case of metric trend lines, there are two layers, but the deleting setup should still act as though only one is present since the second is hidden.

metric-bug

@drewdaemon drewdaemon added bug Fixes for quality problems that affect the customer experience Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Lens labels Oct 19, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors)

@stratoula
Copy link
Contributor

Thing missing is the icon change. I am dropping my assignment here (I fixed the error and now it works ok) as I am going to be out next week and someone else can pick it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
3 participants