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

Update panel styling to improve readability #186

Closed
mcarrano opened this issue Oct 10, 2017 · 11 comments
Closed

Update panel styling to improve readability #186

mcarrano opened this issue Oct 10, 2017 · 11 comments

Comments

@mcarrano
Copy link

Use of saturated panel background colors to reflect status on current background results in poor overall readability. Recommend updating panel json for all Tendrl dashboards according to the following guidance to improve readability per Grafana panel type.

1- SingleStat panels - use text coloring to reflect state, where applicable, i.e. "Healthy" = green; "Unhealthy = orange"

2- Vonage Status Panels (used for multiple, stacked values) - set 'ColorMode = Metric' to use text rather than background fill to reflect status.

3- Table panels - set 'ColorMode' = Value to use text rather that background fill to reflect status.

The attached screen shot shows how this will look when applied to the Tendrl-Cluster-At-A-Glance dashboard. Note that the background for all panels is light gray. I have also attached the json of this dashboard for reference.

screen shot 2017-10-10 at 5 14 39 pm

Tendrl-Gluster-at-a-Glance Copy-1507669380270.txt

@mcarrano
Copy link
Author

@julienlim FYI

@cloudbehl
Copy link
Member

cloudbehl commented Oct 11, 2017

@mcarrano I think this is a good Idea and Surely it will improve the readability of the dashboard.

We can change the coloring for the single stat and the Vonage Status plugin panel to improve the readability. But in the table panel, we are indicating the status which needs to be visually bold because the number doesn't make much sense in that case. So for the table panel, we can go with the current pattern.

@mcarrano @julienlim @nthomas-redhat @brainfunked

@nthomas-redhat
Copy link
Contributor

@cloudbehl , looks good to me

cloudbehl added a commit to cloudbehl/monitoring-integration that referenced this issue Oct 11, 2017
@cloudbehl cloudbehl mentioned this issue Oct 11, 2017
@cloudbehl
Copy link
Member

@mcarrano made the changes. Please have a look.
screenshot from 2017-10-11 14-48-24

@julienlim
Copy link
Member

@cloudbehl @mcarrano - looks good. I'll look forward to seeing these changes in all the dashboards wherever applicable.

@julienlim
Copy link
Member

julienlim commented Oct 11, 2017

@cloudbehl @mcarrano @nthomas-redhat @Tendrl/tendrl-qe
I just redeployed with tendrl-ansible using master, and the styling updates for the singlestat panels and status panels look good!

screen shot 2017-10-11 at 1 32 13 pm

screen shot 2017-10-11 at 1 33 51 pm

screen shot 2017-10-11 at 1 40 13 pm

screen shot 2017-10-11 at 1 40 24 pm

@julienlim
Copy link
Member

julienlim commented Oct 11, 2017

@cloudbehl @mcarrano I just made one of my nodes in the cluster go down, and noticed that the Cluster Status changed to red (unhealthy). It should be orange (unhealthy).

screen shot 2017-10-11 at 2 42 39 pm

This needs to be synchronized with also Tendrl UI. There's a related bug at Tendrl/ui#673 (comment).

I also noted that the tables (table panels) at the bottom still have the background colors there vs. the text color change. Can we change it per @mcarrano's suggestion above. If we can change it so the entire row text color changes to the color used for the status, it should look less overpowering but still provide enough visual cues to the user.

screen shot 2017-10-11 at 2 58 56 pm

@mcarrano
Copy link
Author

Looks so much better @cloudbehl Thanks for making these changes.

@julienlim
Copy link
Member

@cloudbehl Per @mcarrano's investigation, he said regarding the table panels that it did not appear to be possible to apply the text color to the entire row as we had hoped.

So, for now let's leave the tables as currently implemented.

Thank you for the quick turnaround in implementing the singlestat and status panel changes!

cloudbehl added a commit to cloudbehl/monitoring-integration that referenced this issue Oct 16, 2017
- Disk IO Panel - put millisecond (or ms) as part of the y-axis
- Volume utilization trend chart fix
- Decimal and Text size Issues
- Cluster Status changed to red (unhealthy). It should be orange (unhealthy)

tendrl-bug-ids: Tendrl#195, Tendrl#190, Tendrl#211, Tendrl#186
@cloudbehl cloudbehl mentioned this issue Oct 16, 2017
@rishubhjain
Copy link
Contributor

@julienlim If possible please verify if the issue is still reproducible after the fix. Please update the this issue as per your verification.

@julienlim
Copy link
Member

Just noting that this "update panel styling to improve readability" bug impacts all the dashboards:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants