Skip to content

[6.x] [ML] Explorer Chart Tweaks (#22955)#22957

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

[6.x] [ML] Explorer Chart Tweaks (#22955)#22957
walterra merged 1 commit intoelastic:6.xfrom
walterra:backport/6.x/pr-22955

Conversation

@walterra
Copy link
Copy Markdown
Contributor

Backports the following commits to 6.x:

- The aim of this is to more clearly visualize how the timerange of the cell selected in the swimlane relates to the time span shown in the charts.
- The most important change is that the vertical date axis ticks no longer are randomly positioned by d3. Instead they are aligned with the cell interval of the swimlane. This way, the date information shown in the swimlane tooltip will always align with the date tick shown left of the emphasized area in the chart.
- The highlighted area now features a gray rounded border to resemble the styling of the selected cell in the swimlane.
- The chart also fixes where to long chart headers would wrap the "View" link to a new line.
- The x/y axis labels blackness has been reduced to reduce emphasis on the labels.
@walterra walterra added the backport This PR is a backport of another PR label Sep 12, 2018
@walterra walterra self-assigned this Sep 12, 2018
@walterra walterra added :ml Feature:ml-results legacy - do not use labels Sep 12, 2018
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@walterra walterra merged commit 7a788a2 into elastic:6.x Sep 12, 2018
@walterra walterra deleted the backport/6.x/pr-22955 branch September 12, 2018 13:07
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