Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Address UX feedback. Issues: #33 #35

Conversation

yizheliu-amazon
Copy link
Contributor

Issue #, if available:
#33
Description of changes:
Address UX feedback:

  1. Move the Create Detector button to align with Dashboard title
  2. Make sure Live Anomalies title is consistent with AnomalyResultLiveChart page
  3. Change the font of "10 detectors with the most recent anomaly occurrence" to that of "Detector with the most recent anomaly occurrence" above
  4. Reduce the height of Live chart by half, in order to allow the user see more content without scrolling down. To be consistent with AnomalyResultLiveChart page, I have checked with Yaliang, since he reduces to 200px, I will reduce height to 200px as well.
  5. Make sure sunburst chart covers 60% width, while detector/feature list covers 40% width(Done by PR 30)
  6. Make sure sunburst chart's space between upper and lower border is only 20px

After fix, page looks like below:
Screen Shot 2020-04-17 at 3 12 26 PM
Screen Shot 2020-04-17 at 3 12 41 PM

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@@ -1,11 +1,38 @@
import React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add copyright header

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will add it.

@yizheliu-amazon yizheliu-amazon merged commit 68fcc00 into opendistro-for-elasticsearch:development Apr 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants