Merge branch 'dev'

app/assets/javascripts/new_design/state_button.js

@@ -1,9 +1,9 @@
-TPS.acceptDossier = function () {
+TPS.showMotivation = function (state) {
-  $(".motivation").show();
+  $(".motivation." + state).show();
   $(".dropdown-items").hide();
-}
+};
 
 TPS.motivationCancel = function () {
   $(".motivation").hide();
   $(".dropdown-items").show();
-}
+};

app/assets/stylesheets/new_design/dossiers_table.scss

@@ -42,8 +42,7 @@
   }
 
   .notification-col {
-    width: 30px;
+    width: 45px;
-    text-align: center;
 
     a {
       font-size: 16px;
@@ -52,7 +51,6 @@
 
   .number-col {
     width: 110px;
-    text-align: center;
   }
 
   .folder-col {
@@ -69,7 +67,7 @@
   }
 
   .follow-col {
-    width: 190px;
+    width: 200px;
     text-align: right;
     padding-left: $default-spacer;
     padding-right: $default-spacer;

app/assets/stylesheets/new_design/motivation.scss

@@ -10,6 +10,7 @@
   h3 {
     font-size: 22px;
     margin-bottom: $default-spacer * 2;
+    line-height: 1.5;
 
     .icon {
       vertical-align: sub;

app/views/new_gestionnaire/dossiers/_state_button.html.haml

@@ -28,36 +28,24 @@
             .description
               %h4 En instruction
               L'usager ne peut modifier son dossier pendant l'instruction
-          %li{ onclick: "TPS.acceptDossier();" }
+          %li{ onclick: "TPS.showMotivation('accept');" }
             .icon.accept
             .description
               %h4 Accepter
               L'usager sera notifié que son dossier a été accepté
-          %li
+          %li{ onclick: "TPS.showMotivation('without-continuation');" }
-            = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "without_continuation", new_ui: true), method: :post, data: { confirm: "Confirmer vous le classement sans suite de ce dossier ?" } do
+            .icon.without-continuation
-              .icon.without-continuation
+            .description
-              .description
+              %h4 Classer sans suite
-                %h4 Classer sans suite
+              L'usager ne recevra aucune notification
-                L'usager ne recevra aucune notification
+          %li{ onclick: "TPS.showMotivation('close');" }
-          %li
+            .icon.close
-            = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "refuse", new_ui: true), method: :post, data: { confirm: "Confirmer vous le refus de ce dossier ?" } do
+            .description
-              .icon.close
+              %h4 Refuser
-              .description
+              L'usager sera notifié que son dossier a été refusé
-                %h4 Refuser
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Accepter le dossier',  popup_class: 'accept', process_action: 'close', title: 'Accepter', confirm: 'Accepter ce dossier ?' }
-                L'usager sera notifié que son dossier a été refusé
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Classer le dossier sans suite', popup_class: 'without-continuation', process_action: 'without_continuation', title: 'Classer sans suite', confirm: 'Confirmer vous le classement sans suite de ce dossier ?' }
-      .motivation
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Refuser le dossier', popup_class: 'close', process_action: 'refuse', title: 'Refuser', confirm: 'Confirmer vous le refus de ce dossier ?' }
-        %h3
-          .icon.accept
-          Accepter le dossier
-          = "nº #{dossier.id}"
-
-        = form_tag(backoffice_dossier_process_dossier_url(dossier.id, new_ui: true), method: :post, class: "form") do
-          = text_area :dossier, :motivation, class: "motivation-text-area", placeholder: "Rédigez votre motivation ici (facultative)"
-          %p.help
-            L'acceptation du dossier envoie automatiquement une attestation à l'usager.
-          .text-right
-            %span.button{ onclick: "TPS.motivationCancel();" } Annuler
-            = button_tag "Valider la décision", name: :process_action, value: "close", class: 'button primary', title: 'Accepter', data: { confirm: "Accepter ce dossier ?" }
 
 - else
   - if dossier.motivation.present? || dossier.attestation.present?

app/views/new_gestionnaire/dossiers/_state_button_motivation.html.haml

@@ -0,0 +1,13 @@
+.motivation{ class: popup_class }
+  %h3
+    .icon{ class: popup_class }
+    #{popup_title} nº #{dossier.id}
+
+  = form_tag(backoffice_dossier_process_dossier_url(dossier.id, new_ui: true), method: :post, class: 'form') do
+    = text_area :dossier, :motivation, class: 'motivation-text-area', placeholder: 'Rédigez votre motivation ici (facultative)'
+    - if title == 'Accepter'
+      %p.help
+        L'acceptation du dossier envoie automatiquement une attestation à l'usager.
+    .text-right
+      %span.button{ onclick: 'TPS.motivationCancel();' } Annuler
+      = button_tag 'Valider la décision', name: :process_action, value: process_action, class: 'button primary', title: title, data: { confirm: confirm }

spec/features/new_gestionnaire/gestionnaire_spec.rb

@@ -23,8 +23,10 @@ feature 'The gestionnaire part' do
     dossier.reload
     expect(dossier.state).to eq('received')
 
-    fill_in('dossier_motivation', with: 'a good reason')
+    within('.accept.motivation') do
-    click_on 'Valider la décision'
+      fill_in('dossier_motivation', with: 'a good reason')
+      click_on 'Valider la décision'
+    end
 
     dossier.reload
     expect(dossier.state).to eq('closed')