Merge pull request #697 from sgmap/better_procedure_links

Procedure Index: increase link size to the whole line and apply same …

app/assets/stylesheets/new_design/procedure_list.scss

@@ -1,14 +1,28 @@
 @import "colors";
+@import "constants";
 @import "mixins";
 
 .procedure-list {
   .procedure-item {
-    @include vertical-padding(24px);
     border-bottom: 1px solid $border-grey;
 
     &:last-child {
       border-bottom: none;
     }
+
+    a {
+      @include vertical-padding(24px);
+      color: $black;
+      width: 100%;
+
+      &:hover {
+        background-color: $light-grey;
+
+        .procedure-title {
+          text-decoration: underline;
+        }
+      }
+    }
   }
 
   .procedure-logo {
@@ -21,14 +35,7 @@
     min-height: 40px;
     font-size: 20px;
     margin-bottom: 16px;
-
+    padding-left: $default-spacer;
-    a {
-      color: $black;
-
-      &:hover {
-        color: $blue;
-      }
-    }
   }
 
   .procedure-status {

app/views/new_gestionnaire/procedures/index.html.haml

@@ -4,47 +4,47 @@
   %ul.procedure-list
     - @procedures.each do |p|
       %li.procedure-item.flex.align-start
+        = link_to(procedure_path(p)) do
+          .procedure-details
+            %p.procedure-title
+              - if p.logo.present?
+                = image_tag p.logo, class: "procedure-logo"
+              #{p.libelle}
 
-        .procedure-details
+            %ul.procedure-stats.flex
-          %p.procedure-title
+              %li
-            - if p.logo.present?
+                - a_suivre_count = @dossiers_a_suivre_count_per_procedure[p.id] || 0
-              = image_tag p.logo, class: "procedure-logo"
+                .stats-number
-            = link_to(p.libelle, procedure_path(p))
+                  = a_suivre_count
+                .stats-legend
+                  à suivre
+              %li
+                - if @notifications_count_per_procedure[p.id].present?
+                  %span.notifications{ 'aria-label': "notifications" }
+                - followed_count = @followed_dossiers_count_per_procedure[p.id] || 0
+                .stats-number
+                  = followed_count
+                .stats-legend
+                  = t('pluralize.followed', count: followed_count)
+              %li
+                - termines_count = @dossiers_termines_count_per_procedure[p.id] || 0
+                .stats-number
+                  = termines_count
+                .stats-legend
+                  = t('pluralize.processed', count: termines_count)
+              %li
+                - dossier_count = @dossiers_count_per_procedure[p.id] || 0
+                .stats-number
+                  = dossier_count
+                .stats-legend
+                  = t('pluralize.case', count: dossier_count)
+              %li
+                - archived_count = @dossiers_archived_count_per_procedure[p.id] || 0
+                .stats-number
+                  = archived_count
+                .stats-legend
+                  = t('pluralize.archived', count: archived_count)
 
-          %ul.procedure-stats.flex
+          - if p.archivee?
-            %li
+            .procedure-status
-              - a_suivre_count = @dossiers_a_suivre_count_per_procedure[p.id] || 0
+              %span.label Archivée
-              .stats-number
-                = a_suivre_count
-              .stats-legend
-                à suivre
-            %li
-              - if @notifications_count_per_procedure[p.id].present?
-                %span.notifications{ 'aria-label': "notifications" }
-              - followed_count = @followed_dossiers_count_per_procedure[p.id] || 0
-              .stats-number
-                = followed_count
-              .stats-legend
-                = t('pluralize.followed', count: followed_count)
-            %li
-              - termines_count = @dossiers_termines_count_per_procedure[p.id] || 0
-              .stats-number
-                = termines_count
-              .stats-legend
-                = t('pluralize.processed', count: termines_count)
-            %li
-              - dossier_count = @dossiers_count_per_procedure[p.id] || 0
-              .stats-number
-                = dossier_count
-              .stats-legend
-                = t('pluralize.case', count: dossier_count)
-            %li
-              - archived_count = @dossiers_archived_count_per_procedure[p.id] || 0
-              .stats-number
-                = archived_count
-              .stats-legend
-                = t('pluralize.archived', count: archived_count)
-
-        - if p.archivee?
-          .procedure-status
-            %span.label Archivée