Service health graphs consistently rendered

Service health graphs consistently rendered

d3 was sorting by the largest value, now sorts in the proper order: error - pending - running

https://code.launchpad.net/~makyo/juju-gui/consistent-health-graph/+merge/130005

(do not edit description out of merge proposal)

[matthew.scott, 2012-10-16 23:03:52]

Please take a look.

[benjamin.saller, 2012-10-17 18:17:47]

LGTM

https://codereview.appspot.com/6718048/diff/1/app/views/environment.js
File app/views/environment.js (right):

https://codereview.appspot.com/6718048/diff/1/app/views/environment.js#newcod...
app/views/environment.js:784: });
This looks fine to me, can we however be sure those are the only three states
bubbling to this layer? I know that is supposed to be the case, but should these
ever come out of sync having a fallback value here might make sense. 

Probably fine for now though

[matthew.scott, 2012-10-17 19:39:45]

*** Submitted:

Service health graphs consistently rendered

d3 was sorting by the largest value, now sorts in the proper order: error -
pending - running

R=benjamin.saller
CC=
https://codereview.appspot.com/6718048

[matthew.scott, 2012-10-17 21:13:59]

*** Submitted:

Service health graphs consistently rendered

d3 was sorting by the largest value, now sorts in the proper order: error -
pending - running

R=
CC=
https://codereview.appspot.com/6718048