sensu / sensu-dashboard

A dashboard for Sensu, for displaying & managing events & clients.
http://sensuapp.org
MIT License
96 stars 44 forks source link

Compressing modal views. #135

Closed nstielau closed 11 years ago

nstielau commented 11 years ago

The current modal view for events is rather long, and hard to quickly scan for information.

This pull request changes two aspects. Firstly, it puts all of the key/value data into a single 'well' div, which makes it much easier to scan. Secondly, it does not show 'nil/false' data elements. Data that evals to false could be valid, but this opts for simplicity and clarity over verbosity (if there are strong feelings I'd remove this to keep the single-well element).

booyeah

joemiller commented 11 years ago

+1, but i'm biased.

Here is a screenshot of what this looks like:

screenshot_5_21_13_12_15_pm-2

amdprophet commented 11 years ago

Just going to test it locally and I'll merge it in.