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

[Metrics UI] Anomaly detection UI cleanup #78904

Closed
katrin-freihofner opened this issue Sep 30, 2020 · 4 comments
Closed

[Metrics UI] Anomaly detection UI cleanup #78904

katrin-freihofner opened this issue Sep 30, 2020 · 4 comments
Assignees
Labels
Feature:Metrics UI Metrics UI feature polish Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@katrin-freihofner
Copy link
Contributor

During a review, I found a couple of things we should improve to polish our UI.

1. Typo

I suggest: Checking status of machine learning jobs
Screenshot 2020-09-29 at 13 35 39

2. Create jobs

I'm not sure why the copy is "Create ML jobs" if there are already jobs?
I suggest to use Machine learning jobs as a headline
Screenshot 2020-09-29 at 13 38 31

Also something like "Anomaly detection is powered by machine learning. Machine learning jobs are available for the following resource types. Enable these jobs to begin detecting anomalies in your infrastructure metrics."

Please use sentence case.

3. Anomalies tab

The anomalies tab currently navigates to the machine learning app. I think a tab that navigates somewhere is pretty unusual.
Screenshot 2020-09-29 at 13 37 36

I understand that this is the first version until the anomalies list is available. That is why I suggest removing the tabs for now and linking to the anomalies list in Machine learning with a button.

I'm thinking about something like this:
Screenshot 2020-09-30 at 08 46 02

4. Anomaly detection button

I suggest aligning the button in the header with the one we have in APM. Which would mean adding an icon and calling it Anomaly detection.

Screenshot 2020-09-30 at 08 48 43

Screenshots from the current Master branch.

@katrin-freihofner katrin-freihofner added Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services polish labels Sep 30, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@sgrodzicki sgrodzicki added this to the Metrics UI 7.10 milestone Sep 30, 2020
@phillipb
Copy link
Contributor

@katrin-freihofner What do you think about this? I just removed the "Create ML Jobs" header completely. I don't think it added much. Also, I changed "Create jobs" to "Enable" since I feel like that's a better call to action. I also, updated the copy in each of the cards correctly reflect the types of metrics that we're detecting anomalies for.

Screen Shot 2020-09-30 at 10 57 17 AM

@katrin-freihofner
Copy link
Contributor Author

I think it's much better. Thank you! One tiny thing, could you please make the button label "Recreate jobs" sentence case.

And the buttons "Manage jobs" and "View anomalies" are not there if a user has ML not enabled, right?

Thank you very much for this fix!

@phillipb
Copy link
Contributor

phillipb commented Oct 5, 2020

Closed by #78875

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Metrics UI Metrics UI feature polish Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

4 participants