add sidemenu for instructeurs instead of links at top of page

app/views/experts/avis/_sidemenu.html.haml

@@ -3,16 +3,19 @@
     %button.fr-sidemenu__btn{"aria-controls" => "fr-sidemenu-wrapper", "aria-expanded" => "false", hidden: ""} Dans cette rubrique
     #fr-sidemenu-wrapper.fr-collapse
       %ul.fr-sidemenu__list
-        - current_page = current_page?(instruction_expert_avis_path(@avis.procedure, @avis))
+        - url = instruction_expert_avis_path(@avis.procedure, @avis)
+        - current_page = current_page?(url)
         %li{class: "fr-sidemenu__item fr-sidemenu__item#{current_page ? '--active' : ''}"}
-          %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil, href: instruction_expert_avis_path(@avis.procedure, @avis), target: "_self"} Donner votre avis
+          %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil, href: url, target: "_self"} Donner votre avis
 
         - if @dossier.avis_for_expert(current_expert).present?
-          - current_page = current_page?(avis_list_expert_avis_path(@avis.procedure, @avis))
+          - url = avis_list_expert_avis_path(@avis.procedure, @avis)
+          - current_page = current_page?(url)
           %li{class: "fr-sidemenu__item fr-sidemenu__item#{current_page ? '--active' : ''}"}
-            %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil,href: avis_list_expert_avis_path(@avis.procedure, @avis), target: "_self"} Voir les avis
+            %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil,href: url, target: "_self"} Voir les avis
 
         - if !@dossier.termine?
-          - current_page = current_page?(avis_new_expert_avis_path(@avis.procedure, @avis))
+          - url = avis_new_expert_avis_path(@avis.procedure, @avis)
+          - current_page = current_page?(url)
           %li{class: "fr-sidemenu__item fr-sidemenu__item#{current_page ? '--active' : ''}"}
-            %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil,href: avis_new_expert_avis_path(@avis.procedure, @avis), target: "_self"} Demander un nouvel avis
+            %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil,href: url, target: "_self"} Demander un nouvel avis

app/views/instructeurs/avis/_sidemenu.html.haml

@@ -0,0 +1,14 @@
+%nav.fr-sidemenu{"aria-labelledby" => "fr-sidemenu-title"}
+  .fr-sidemenu__inner
+    %button.fr-sidemenu__btn{"aria-controls" => "fr-sidemenu-wrapper", "aria-expanded" => "false", hidden: ""} Dans cette rubrique
+    #fr-sidemenu-wrapper.fr-collapse
+      %ul.fr-sidemenu__list
+        - url = avis_new_instructeur_dossier_path(@dossier.procedure, @dossier)
+        - current_page = current_page?(url)
+        %li{class: "fr-sidemenu__item fr-sidemenu__item#{current_page ? '--active' : ''}"}
+          %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil, href: url, target: "_self"} Demander un avis
+
+        - url = avis_instructeur_dossier_path(@dossier.procedure, @dossier)
+        - current_page = current_page?(url)
+        %li{class: "fr-sidemenu__item fr-sidemenu__item#{current_page ? '--active' : ''}"}
+          %a.fr-sidemenu__link{'aria-current': current_page ? 'page' : nil,href: url, target: "_self"} Voir les avis

app/views/instructeurs/dossiers/avis.html.haml

@@ -3,19 +3,13 @@
 = render partial: "header", locals: { dossier: @dossier }
 
 .container
-  - if !@dossier.termine?
-    - if @dossier.procedure.allow_expert_review
+  .fr-grid-row
+    .fr-col.fr-col-12.fr-col-md-3
+      = render partial: 'instructeurs/avis/sidemenu'
+    .fr-col
       - if @dossier.avis.present?
-        = link_to 'demander un nouvel avis', avis_new_instructeur_dossier_path(@dossier.procedure, @dossier), class: 'fr-btn fr-btn--sm pull-right'
-      - else
-        = render partial: "instructeurs/avis/shared/form", locals: { url: avis_instructeur_dossier_path(@dossier.procedure, @dossier), linked_dossiers: @dossier.linked_dossiers_for(current_instructeur), must_be_confidentiel: false, avis: @avis }
-    - else
-      %p Cette démarche n’autorise pas la demande d’avis à un expert. Veuillez contacter votre administrateur
+        = render partial: 'instructeurs/avis/shared/list', locals: { avis: @dossier.avis, avis_seen_at: @avis_seen_at }
 
-  - if @dossier.avis.present?
-    = render partial: 'instructeurs/avis/shared/list', locals: { avis: @dossier.avis, avis_seen_at: @avis_seen_at }
-
-  - if @dossier.termine? && !@dossier.avis.present?
-    .blank-tab
-      %h2.empty-text Aucun avis.
-      %p.empty-text-details Aucun avis n’a été demandé sur ce dossier.
+      -else
+        %h2.empty-text Aucun avis.
+        %p.empty-text-details Aucun avis n’a été demandé sur ce dossier.

app/views/instructeurs/dossiers/avis_new.html.haml

@@ -3,10 +3,12 @@
 = render partial: "header", locals: { dossier: @dossier }
 
 .container
-  - if !@dossier.termine?
-    - if @dossier.procedure.allow_expert_review
-      - if @dossier.avis.present?
-        = link_to '< retour aux avis', avis_instructeur_dossier_path(@dossier.procedure, @dossier), class: 'fr-link'
-      = render partial: "instructeurs/avis/shared/form", locals: { url: avis_instructeur_dossier_path(@dossier.procedure, @dossier), linked_dossiers: @dossier.linked_dossiers_for(current_instructeur), must_be_confidentiel: false, avis: @avis }
-    - else
-      %p Cette démarche n’autorise pas la demande d’avis à un expert. Veuillez contacter votre administrateur
+  .fr-grid-row
+    .fr-col.fr-col-12.fr-col-md-3
+      = render partial: 'instructeurs/avis/sidemenu'
+    .fr-col
+      - if !@dossier.termine?
+        - if @dossier.procedure.allow_expert_review
+          = render partial: "instructeurs/avis/shared/form", locals: { url: avis_instructeur_dossier_path(@dossier.procedure, @dossier), linked_dossiers: @dossier.linked_dossiers_for(current_instructeur), must_be_confidentiel: false, avis: @avis }
+        - else
+          %p Cette démarche n’autorise pas la demande d’avis à un expert. Veuillez contacter votre administrateur