Skip to content

[7.x] [ML] DF Analytics: Collapsable sections on results pages (#76641)#79188

Merged
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-76641
Oct 2, 2020
Merged

[7.x] [ML] DF Analytics: Collapsable sections on results pages (#76641)#79188
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-76641

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented Oct 1, 2020

Backports the following commits to 7.x:

- Fixes cell color coding based on influence score for outlier detection results page data grid. (Part of elastic#77046)
- Introduces expandable sections (<ExpandableSection />). In contrast to plain accordions, the main idea of this component is that it should also provide some sort of useful summary when collapsed instead of just being an expandable title. For example, the "Analysis" section is collapsed by default, but still offers information like analysis type, source and destination index. This concept should allow us to keep the analytics results pages usable with more content (additional results, evaluations, visualizations) being added over time.
- The "Analysis" section is a reuse of the expandable row from the analytics jobs list. Some design adjustments have been made to make it usable in both places.
@walterra walterra added the backport This PR is a backport of another PR label Oct 1, 2020
@walterra walterra self-assigned this Oct 1, 2020
@walterra walterra added the :ml label Oct 1, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
ml 1421 1450 +29

async chunks size

id before after diff
ml 8.9MB 8.9MB +42.8KB

page load bundle size

id before after diff
ml 734.8KB 734.9KB +131.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@walterra walterra merged commit ec6fd9e into elastic:7.x Oct 2, 2020
@walterra walterra deleted the backport/7.x/pr-76641 branch October 2, 2020 06:27
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 :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants