Skip to content

Comments

[6.x] [ML] Fix reloading anomaly charts on resize. (#22967)#22995

Merged
walterra merged 1 commit intoelastic:6.xfrom
walterra:backport/6.x/pr-22967
Sep 13, 2018
Merged

[6.x] [ML] Fix reloading anomaly charts on resize. (#22967)#22995
walterra merged 1 commit intoelastic:6.xfrom
walterra:backport/6.x/pr-22967

Conversation

@walterra
Copy link
Contributor

Backports the following commits to 6.x:

- Fixes a regression introduced in elastic#22814. Because of the stricter checking for scope/props updates, resizing the browser window would miss updating the Anomaly Explorer Charts widths. This fixes it by adding a check to trigger anomalyDataChange in redrawOnResize().
- Additionally, if only one chart is up for display, this update makes sure a single chart always spans across the full available width.
@walterra walterra added the backport This PR is a backport of another PR label Sep 13, 2018
@walterra walterra self-assigned this Sep 13, 2018
@walterra walterra added :ml Feature:ml-results legacy - do not use labels Sep 13, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@walterra walterra merged commit ecfbfa7 into elastic:6.x Sep 13, 2018
@walterra walterra deleted the backport/6.x/pr-22967 branch September 13, 2018 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Feature:ml-results legacy - do not use :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants