From c69d79ab03e59389b1ad527024097216c8f1edf5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Xavier=20Juli=C3=A1n?= <xavier.julian@kaleidos.net>
Date: Tue, 1 Jul 2014 12:52:45 +0200
Subject: [PATCH] Make modificable elements more visible

---
 app/coffee/modules/common/directives.coffee   |  5 +--
 app/coffee/modules/taskboard/main.coffee      |  7 +++-
 .../views/components/backlog-row.jade         |  5 +--
 .../views/modules/taskboard-table.jade        |  2 ++
 app/styles/modules/taskboard-table.scss       | 33 +++++++++++++++----
 5 files changed, 41 insertions(+), 11 deletions(-)

diff --git a/app/coffee/modules/common/directives.coffee b/app/coffee/modules/common/directives.coffee
index 9972268f..b6993084 100644
--- a/app/coffee/modules/common/directives.coffee
+++ b/app/coffee/modules/common/directives.coffee
@@ -100,9 +100,10 @@ UsStatusDirective = ($repo) ->
     """)
 
     updateUsStatus = ($el, us, usStatusById) ->
-        usStatusDom = $el.find(".us-status")
+        usStatusDomParent = $el.find(".us-status")
+        usStatusDom = $el.find(".us-status .us-status-bind")
         usStatusDom.text(usStatusById[us.status].name)
-        usStatusDom.css('color', usStatusById[us.status].color)
+        usStatusDomParent.css('color', usStatusById[us.status].color)
 
     link = ($scope, $el, $attrs) ->
         $ctrl = $el.controller()
diff --git a/app/coffee/modules/taskboard/main.coffee b/app/coffee/modules/taskboard/main.coffee
index 20718b00..0e978c48 100644
--- a/app/coffee/modules/taskboard/main.coffee
+++ b/app/coffee/modules/taskboard/main.coffee
@@ -214,7 +214,11 @@ TaskboardUsPointsDirective = ($repo, $confirm) ->
     pointsTemplate = _.template("""
     <% _.each(usRolePoints, function(rolePoint) { %>
     <li>
-        <%- rolePoint.role.name %> <span class="us-role-points"> <%- rolePoint.point.name %></span>
+        <%- rolePoint.role.name %>
+        <a href="" class="us-role-points">
+            <%- rolePoint.point.name %>
+            <span class="icon icon-arrow-bottom"></span>
+        </a>
         <ul class="popover pop-points">
             <% _.each(points, function(point) { %>
             <li>
@@ -252,6 +256,7 @@ TaskboardUsPointsDirective = ($repo, $confirm) ->
 
         $el.on "click", ".us-role-points", (event) ->
             event.stopPropagation()
+            event.preventDefault()
 
             target = angular.element(event.currentTarget)
             popover = target.parent().find(".pop-points")
diff --git a/app/partials/views/components/backlog-row.jade b/app/partials/views/components/backlog-row.jade
index d83519ea..93c0931a 100644
--- a/app/partials/views/components/backlog-row.jade
+++ b/app/partials/views/components/backlog-row.jade
@@ -10,8 +10,9 @@ div.row.us-item-row(ng-repeat="us in visibleUserstories track by us.id")
             span.tag(ng-repeat="tag in us.tags") {{ tag }}
 
     div.status.width-2(tg-us-status="us" on-update="ctrl.loadProjectStats()")
-       a.us-status(href="", title="Status Name")
-           span.icon.icon-arrow-bottom
+        a.us-status(href="", title="Status Name")
+            span.us-status-bind
+            span.icon.icon-arrow-bottom
 
     div.points(tg-us-points="us")
         a.us-points(href="", title="Points")
diff --git a/app/partials/views/modules/taskboard-table.jade b/app/partials/views/modules/taskboard-table.jade
index 5d753849..e9d88f40 100644
--- a/app/partials/views/modules/taskboard-table.jade
+++ b/app/partials/views/modules/taskboard-table.jade
@@ -14,6 +14,8 @@ div.taskboard-table
                         span(ng-bind="us.subject")
                     div.status(tg-us-status="us", on-update="ctrl.loadSprintState()")
                         a.us-status(href="", title="Status Name")
+                            span.us-status-bind
+                            span.icon.icon-arrow-bottom
                     ul.points-list(tg-taskboard-us-points="us")
                     include ../components/addnewtask
 
diff --git a/app/styles/modules/taskboard-table.scss b/app/styles/modules/taskboard-table.scss
index eef85068..29906778 100644
--- a/app/styles/modules/taskboard-table.scss
+++ b/app/styles/modules/taskboard-table.scss
@@ -75,16 +75,20 @@ $column-margin: 0 10px 0 0;
         color: $green-taiga;
         margin-bottom: .5rem;
         position: relative;
+        .icon {
+            @include transition (opacity .2s linear);
+            opacity: 0;
+        }
+        &:hover {
+            .icon {
+                @include transition (opacity .2s linear);
+                opacity: 1;
+            }
+        }
     }
     .pop-status {
         @include popover(150px, 20px, 30px, '', '');
     }
-    .points-list {
-        span {
-            color: $gray;
-            margin-left: .5rem;
-        }
-    }
     .pop-points {
         @include popover(200px, 125px, 30px, '', '');
         li {
@@ -108,3 +112,20 @@ $column-margin: 0 10px 0 0;
         padding: 7px 80px;
     }
 }
+
+.points-list {
+    a {
+        color: $green-taiga;
+        margin-left: .5rem;
+        &:hover {
+            span {
+                @include transition (opacity .2s linear);
+                opacity: 1;
+            }
+        }
+    }
+    span {
+        @include transition (opacity .2s linear);
+        opacity: 0;
+    }
+}