From 172b8cfb9b0f20fb91eea6b0fbaaf6e39f9cd46f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ga=C3=ABl=20M=C3=A9tais?= <gael.metais@gmail.com>
Date: Fri, 31 Jul 2015 19:41:47 +0200
Subject: [PATCH] Change ng-click to real links on rule rows

---
 front/src/css/dashboard.css    | 10 +++++++---
 front/src/css/timeline.css     |  6 ++++--
 front/src/less/dashboard.less  |  6 +++++-
 front/src/less/timeline.less   |  6 ++++--
 front/src/views/dashboard.html |  6 +++---
 5 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/front/src/css/dashboard.css b/front/src/css/dashboard.css
index ca1c26f..104695f 100644
--- a/front/src/css/dashboard.css
+++ b/front/src/css/dashboard.css
@@ -109,14 +109,18 @@
 .summary .notations .criteria .table {
   width: 100%;
 }
-.summary .notations .criteria .table > div:hover > div {
+.summary .notations .criteria .table > a {
+  text-decoration: none;
+  color: inherit;
+}
+.summary .notations .criteria .table > a:hover > div {
   background: #EBD8E2;
   cursor: pointer;
 }
-.summary .notations .criteria .table > div:hover > div.info {
+.summary .notations .criteria .table > a:hover > div.info {
   background: #FFF;
 }
-.summary .notations .criteria .table > div:hover > div.info .icon-question {
+.summary .notations .criteria .table > a:hover > div.info .icon-question {
   color: #EBD8E2;
 }
 .summary .notations .criteria .grade {
diff --git a/front/src/css/timeline.css b/front/src/css/timeline.css
index 9ecf796..5b2c872 100644
--- a/front/src/css/timeline.css
+++ b/front/src/css/timeline.css
@@ -152,14 +152,16 @@
   width: 100%;
   border-spacing: 0.25em;
 }
-.table > div {
+.table > div,
+.table > a {
   display: table-row;
 }
 .table > .headers > div {
   font-weight: bold;
   padding: 0.5em 1em;
 }
-.table > div > div {
+.table > div > div,
+.table > a > div {
   padding: 0.1em 1em;
   background: #f2f2f2;
   display: table-cell;
diff --git a/front/src/less/dashboard.less b/front/src/less/dashboard.less
index 1eeaaa5..b46ed02 100644
--- a/front/src/less/dashboard.less
+++ b/front/src/less/dashboard.less
@@ -108,7 +108,11 @@
 
 .summary .notations .criteria .table {
     width: 100%;
-    > div:hover > div {
+    > a {
+        text-decoration: none;
+        color: inherit;
+    }
+    > a:hover > div {
         background: #EBD8E2;
         cursor: pointer;
         &.info {
diff --git a/front/src/less/timeline.less b/front/src/less/timeline.less
index ab9bd80..d59beac 100644
--- a/front/src/less/timeline.less
+++ b/front/src/less/timeline.less
@@ -173,7 +173,8 @@
     border-spacing: 0.25em;
 }
 
-.table > div {
+.table > div,
+.table > a {
     display: table-row;
 }
 
@@ -182,7 +183,8 @@
     padding: 0.5em 1em;
 }
 
-.table > div > div {
+.table > div > div,
+.table > a > div {
     padding: 0.1em 1em;
     background: #f2f2f2;
     display: table-cell;
diff --git a/front/src/views/dashboard.html b/front/src/views/dashboard.html
index 5efffa0..a7a2191 100644
--- a/front/src/views/dashboard.html
+++ b/front/src/views/dashboard.html
@@ -28,8 +28,8 @@
             <div class="category">{{category.label}}</div>
             <div class="criteria">
                 <div class="table" title="Click to see details">
-                    <div ng-repeat="ruleName in category.rules" ng-if="result.rules[ruleName]" ng-init="rule = result.rules[ruleName]"
-                         ng-class="{'warning': rule.abnormal}" ng-click="showRulePage(ruleName)">
+                    <a ng-repeat="ruleName in category.rules" ng-if="result.rules[ruleName]" ng-init="rule = result.rules[ruleName]"
+                         ng-class="{'warning': rule.abnormal}" href="/result/{{runId}}/rule/{{ruleName}}">
                         <div class="grade">
                             <grade score="rule.score"></grade>
                         </div>
@@ -42,7 +42,7 @@
                             <span ng-if="rule.abnormalityScore <= -300" class="icon-warning"></span>
                         </div>
                         <div class="info"><span class="icon-question"></span></div>
-                    </div>
+                    </a>
                 </div>
             </div>
         </div>