From 937095869363a442de89db49b8a305515882f3f0 Mon Sep 17 00:00:00 2001
From: Tyler Renelle <tylerrenelle@gmail.com>
Date: Mon, 6 May 2013 17:57:06 +0100
Subject: [PATCH] tags: move tags stuff in tasks html out to filters.html,
 click the tag on a task to apply tag

---
 styles/app/filters.styl |  6 +++++-
 views/app/filters.html  | 22 +++++++++++++++++++++-
 views/app/tasks.html    | 19 ++-----------------
 3 files changed, 28 insertions(+), 19 deletions(-)

diff --git a/styles/app/filters.styl b/styles/app/filters.styl
index b9dbced5b1..d734addab7 100644
--- a/styles/app/filters.styl
+++ b/styles/app/filters.styl
@@ -12,4 +12,8 @@
 
 /* Filters applied to tasks */
 .task.filtered-out
-  display: none
\ No newline at end of file
+  display: none
+
+.tag-label:hover
+  background-color: #3a87ad
+  cursor: pointer
\ No newline at end of file
diff --git a/views/app/filters.html b/views/app/filters.html
index 24ef93d1e3..5efb12b288 100644
--- a/views/app/filters.html
+++ b/views/app/filters.html
@@ -30,4 +30,24 @@
           {/}
       </li>
     </ul>
-  </div>
\ No newline at end of file
+  </div>
+
+<applied-filters:>
+    <!-- There's a very strange bug where live-bound #each here loses model.at reference on _{type}List alter -->
+    {{#each _user.tags as :tag}}
+        <span>
+            {#if :task.tags[:tag.id]}
+                <span data-tag-id="{{:tag.id}}" x-bind="click:toggleFilterByTag" class="label tag-label {#if _user.filters[:tag.id]}label-info{/}">{:tag.name}</span>
+            {/}
+        </span>
+    {{/}}
+
+<filter-fieldgroup:>
+    <fieldset class='option-group'>
+        <legend class="option-title">Tags</legend>
+        <select multiple="multiple">
+            {{#each _user.tags as :tag}}
+                <option selected="{:task.tags[:tag.id]}">{:tag.name}</option>
+            {{/}}
+        </select>
+    </fieldset>
\ No newline at end of file
diff --git a/views/app/tasks.html b/views/app/tasks.html
index 92e8f7e8df..14f37559aa 100644
--- a/views/app/tasks.html
+++ b/views/app/tasks.html
@@ -152,15 +152,7 @@
         <!-- main content -->
         <p class="task-text">
             {:task.text}
-
-            <!-- There's a very strange bug where live-bound #each here loses model.at reference on _{type}List alter -->
-            {{#each _user.tags as :tag}}
-                <span>
-                    {#if :task.tags[:tag.id]}
-                        <span class="label {#if _user.filters[:tag.id]}label-info{/}">{:tag.name}</span>
-                    {/}
-                </span>
-            {{/}}
+            <app:filters:applied-filters />
         </p>
 
         <!-- edit/options dialog -->
@@ -225,14 +217,7 @@
             {/}
 
 
-            <fieldset class='option-group'>
-                <legend class="option-title">Tags</legend>
-                <select multiple="multiple">
-                    {{#each _user.tags as :tag}}
-                        <option selected="{:task.tags[:tag.id]}">{:tag.name}</option>
-                    {{/}}
-                </select>
-            </fieldset>
+            <app:filters:filter-fieldgroup />
 
 
             <!-- Advanced Options -->