Github user bmahler commented on a diff in the pull request:

    https://github.com/apache/mesos/pull/233#discussion_r142247472
  
    --- Diff: src/webui/master/static/js/app.js ---
    @@ -88,6 +88,14 @@
             return state.substring(5);
           };
         })
    +    .filter('taskHealth', function() {
    +      return function(health) {
    +        if (health != null) {
    +           return health ? "healthy" : "unhealthy";
    --- End diff --
    
    2 space indent, would also be good to warn the reader here that this value 
is relied on for the CSS


---

Reply via email to