small adjustments from PR feedbacks

app/components/dossiers/accuse_lecture_component/accuse_lecture_component.fr.yml

@@ -1,4 +1,4 @@
 ---
 fr:
-  text_accuse_lecture: Cette procédure est soumise à un accusé de lecture. En demandant l'affichage de la décision prise sur votre dossier, vous acceptez l'accusé de lecture et par là même le démarrage du délai légal en cas de recours.
+  text_accuse_lecture: Cette démarche est soumise à un accusé de lecture. En demandant l’affichage de la décision prise sur votre dossier, vous acceptez l’accusé de lecture et par là même le démarrage du délai légal en cas de recours.
   btn_accuse_lecture: Je souhaite afficher la décision

app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml

@@ -4,7 +4,7 @@
       - if @procedure.accuse_lecture.present?
         %p.fr-badge.fr-badge--success Activé
       - else
-        %p.fr-badge.fr-badge--info À configurer
+        %p.fr-badge.fr-badge--info Désactivé
       %div
         %h3.fr-h6.fr-mt-10v= t('.title')
         %p.fr-tile-subtitle= t('.subtitle')

app/helpers/dossier_helper.rb

@@ -80,10 +80,7 @@ module DossierHelper
 
   def status_badge_user(dossier, alignment_class = '')
     if dossier.hide_info_with_accuse_lecture?
-      tag.span 'traité', role: 'status', class: class_names(
-        'fr-badge fr-badge--sm fr-badge--no-icon' => true,
-        alignment_class => true
-      )
+      tag.span 'traité', role: 'status', class: "fr-badge fr-badge--sm fr-badge--no-icon #{alignment_class}"
     else
       status_badge(dossier.state, alignment_class)
     end

app/views/administrateurs/procedures/accuse_lecture.html.haml

@@ -9,10 +9,16 @@
       %h1.page-title
         Accusé de lecture
 
-      = render Dsfr::AlertComponent.new(state: :info, size: :sm, extra_class_names: 'fr-mb-2w') do |c|
+      = render Dsfr::CalloutComponent.new(title: nil) do |c|
         - c.with_body do
           %p
-            Pour les démarches avec voies de recours, il est possible d'activer l'accusé de lecture. L'usager n'a plus accès à la décision finale par mail, mais il doit se connecter sur la plateforme #{Current.application_name} pour en prendre connaissance et en accuser lecture.
+            Pour les démarches avec voies de recours, il est possible d’activer l’accusé de lecture.
+          %br
+          %p
+            Cette fonctionnalité permet à l’instructeur de connaître la date de lecture de la décision finale par l’usager.
+          %br
+          %p
+            L’usager n’a plus accès à la décision finale par mail, mais il doit se connecter sur la plateforme #{Current.application_name} pour en prendre connaissance et en accuser lecture.
 
       %ul.fr-toggle__list
         %li
@@ -24,7 +30,7 @@
             = render Dsfr::ToggleComponent.new(form: f,
               target: :accuse_lecture,
               title: "Accusé de lecture de la démarche",
-              hint: "L’accusé de lecture est à activer uniquement pour les démarches avec voies de recours car il complexifie l'accès à la décision finale pour les usagers",
+              hint: "L’accusé de lecture est à activer uniquement pour les démarches avec voies de recours car il complexifie l’accès à la décision finale pour les usagers",
               opt: {"checked" => @procedure.accuse_lecture})
 
 .padded-fixed-footer

app/views/notification_mailer/send_accuse_lecture_notification.html.haml

@@ -14,10 +14,10 @@
   = t("layouts.mailers.accuse_lecture.second_part")
 %p
   = t("layouts.mailers.accuse_lecture.third_part")
-  = link_to 'Démarches Simplifiées.', dossier_url(@dossier)
+  = link_to APPLICATION_NAME, dossier_url(@dossier)
 
 %p
   = t(:best_regards, scope: [:views, :shared, :greetings])
   %br
     = t('layouts.mailers.signature.team')
-    #{APPLICATION_NAME.gsub(".","⁠.").html_safe}
+    = APPLICATION_NAME

app/views/users/dossiers/show/_status_overview.html.haml

@@ -72,7 +72,7 @@
 
             - if dossier.attestation.present?
               .action
-                = link_to attestation_dossier_path(dossier), class: "fr-btn fr-icon-download-line fr-btn--icon-left", target: '_blank', rel: 'noopener' do
+                = link_to attestation_dossier_path(dossier), class: "fr-btn fr-icon-download-line fr-btn--icon-left", **external_link_attributes do
                   = t('views.users.dossiers.show.status_overview.accepte_attestation')
 
 

spec/views/users/dossiers/demande.html.haml_spec.rb

@@ -64,7 +64,7 @@ describe 'users/dossiers/demande', type: :view do
     let(:dossier) { create(:dossier, :accepte, :with_motivation) }
 
     it 'displays the motivation' do
-      expect(rendered).not_to have_text('Cette procédure est soumise à un accusé de lecture.')
+      expect(rendered).not_to have_text('Cette démarche est soumise à un accusé de lecture.')
       expect(rendered).to have_text('Motivation')
     end
   end
@@ -73,7 +73,7 @@ describe 'users/dossiers/demande', type: :view do
     let(:dossier) { create(:dossier, :accepte, :with_motivation, procedure: create(:procedure, :accuse_lecture)) }
 
     it 'display information about accuse de lecture and not the motivation' do
-      expect(rendered).to have_text('Cette procédure est soumise à un accusé de lecture.')
+      expect(rendered).to have_text('Cette démarche est soumise à un accusé de lecture.')
       expect(rendered).not_to have_text('Motivation')
     end
   end

spec/views/users/dossiers/show/_status_overview.html.haml_spec.rb

@@ -98,6 +98,6 @@ describe 'users/dossiers/show/_status_overview', type: :view do
 
     it { is_expected.not_to have_selector('.status-explanation .sans-suite') }
     it { is_expected.not_to have_text(dossier.motivation) }
-    it { is_expected.to have_text('Cette procédure est soumise à un accusé de lecture.') }
+    it { is_expected.to have_text('Cette démarche est soumise à un accusé de lecture.') }
   end
 end