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

[Alerting] index threshold graph obscured by expression editor #59384

Closed
pmuellr opened this issue Mar 4, 2020 · 4 comments · Fixed by #59399
Closed

[Alerting] index threshold graph obscured by expression editor #59384

pmuellr opened this issue Mar 4, 2020 · 4 comments · Fixed by #59399
Assignees
Labels
Feature:Alerting Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.7.0

Comments

@pmuellr
Copy link
Member

pmuellr commented Mar 4, 2020

In the image below, you can see that I've clicked on the "FOR THE LAST " expression editor bit, to change the integer value. Sadly, that particular editor - and the "IS ABOVE" one next to it, obscure the visualization graph. Ironically, this particular value - we call it the "timeWindow" of the alert, is the one that can change the alert values calculated fairly significantly, which shows in the graph, so you really want to see the graph as this value changes.

threshold

Side note that the graph height shrinks to 0 (or whatever) while the new data is being retrieved, which causes the bits normally hidden "below the fold" to pop up for a bit, before the graph is re-rendered. I guess we need to reserve that space while there isn't a graph being displayed (separate issue, I'd guess).

@pmuellr pmuellr added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Mar 4, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@pmuellr
Copy link
Member Author

pmuellr commented Mar 4, 2020

cc: @andreadelrio @mdefazio

@snide snide added the Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. label Mar 4, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-design (Team:Design)

@snide
Copy link
Contributor

snide commented Mar 4, 2020

Suggestion on this one would be to use anchorPosition to point the popover for the lower row up and to the left. It won't fix every usage, but should fix the majority of them.

I'd also suggest setting a wrapping div with a height of the graph to exist around it. That way we won't get the pop-in rendering. Better yet, put an empty state in there for the chart.

@mikecote mikecote added the v7.7.0 label Mar 9, 2020
@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.7.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants