[Sheriff-o-matic] Refactor cards and get rid of ct-failure-cards

Tools/GardeningServer/ui/ct-failure-stream.html

Index: Tools/GardeningServer/ui/ct-failure-stream.html
diff --git a/Tools/GardeningServer/ui/ct-failure-stream.html b/Tools/GardeningServer/ui/ct-failure-stream.html
index ff2df73de09adae29b22261672ee26b892e04c74..ac1f7101fb5c1433cc159e2d1ef3456b8922c979 100644
--- a/Tools/GardeningServer/ui/ct-failure-stream.html
+++ b/Tools/GardeningServer/ui/ct-failure-stream.html
@@ -4,7 +4,9 @@ Use of this source code is governed by a BSD-style license that can be
 found in the LICENSE file.
 -->
 
-<link rel="import" href="ct-failure-card.html">
+<link rel="import" href="ct-bot-failure-card.html">
+<link rel="import" href="ct-builder-failure-card.html">
+<link rel="import" href="ct-trooper-card.html">
 
 <polymer-element name="ct-failure-stream" attributes="groups commitLog tree category title" noscript>
   <template>
@@ -13,14 +15,15 @@ found in the LICENSE file.
         display: block;
       }
 
-      ct-failure-card {
+      .card {
         margin-bottom: 5px;
         padding: 5px;
         border-bottom: 1px solid lightgrey;
+        display: flex;
       }
 
       /* FIXME: Don't use before hax to add labels */
-      ct-failure-card:first-of-type::before {
+      :host>div:first-of-type::before {

[ojan, 2014/09/11 01:50:04]

Spaced around the >

         content: "{{ title }}:";
         display: block;
         font-weight: bold;
@@ -29,10 +32,27 @@ found in the LICENSE file.
         background-color: #f5f5f5;
         margin-bottom: 5px;
       }
+
+      .snoozed {
+        opacity: 0.5;
+      }
     </style>
     <template repeat="{{ group in groups }}">
       <template if="{{ group.category == category }}">
-        <ct-failure-card group="{{ group }}" commitLog="{{ commitLog }}" tree="{{ tree }}"></ct-failure-card>
+        <div> <!-- FIXME: Remove when we have a better title solution. -->
+          <div class="card">
+            <template if="{{ group.data.category == 'sheriff' }}">
+              <ct-bot-failure-card class='{{ { snoozed: group.isSnoozed } | tokenList }}' group="{{ group.data }}" commitLog="{{ commitLog }}" builderList="{{ _builderList }}"></ct-bot-failure-card>

[ojan, 2014/09/11 01:50:04]

You moved the builder list down into the widget but didn't remove it here. But,
actually, I think the builder list should just be a property on group.data. As a
side benefit, that would also fix a bug where we don't update the builder list
properly when just the builder list changes and nothing else in the failure
group does.

+            </template>
+            <template if="{{ group.data.category == 'builder' }}">
+              <ct-builder-failure-card class='{{ { snoozed: group.isSnoozed } | tokenList }}' group="{{ group.data }}" builderList="{{ _builderList }}"></ct-builder-failure-card>
+            </template>
+            <template if="{{ group.data.category == 'trooper' }}">
+              <ct-trooper-card class='{{ { snoozed: group.isSnoozed } | tokenList }}' group="{{ group.data }}"></ct-trooper-card>
+            </template>
+            <ct-failure-card-buttons group="{{ group }}" bug="{{ bug }}"></ct-failure-card-buttons>
+          </div>
+        </div>
       </template>
     </template>
   </template>