Merge pull request #9146 from demarches-simplifiees/update-ui-when-adding-group

Mise a jour de l'UX de la page groupe d'instructeurs

app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml

@@ -1,5 +1,5 @@
 %div{ id: dom_id(@groupe_instructeur, :routing) }
-  %h2 Paramètres principaux
+  %h1 Paramètres du groupe
 
   = form_for @groupe_instructeur,
     url: admin_procedure_groupe_instructeur_path(@procedure_id, @groupe_instructeur),
@@ -31,26 +31,29 @@
       - if @groupe_instructeur.routing_to_configure?
         %p.fr-mb-1w.fr-badge.fr-badge--warning.fr-badge--sm à configurer
 
-    .flex.align-baseline
+    .flex.align-baseline.fr-mb-1w
       .fr-mr-2w.no-wrap si le champ
       .target.fr-mr-2w
         = targeted_champ_tag
       .operator.fr-mr-2w.no-wrap est égal à
       .value
         = value_tag
+    .fr-hint-text
+      %span Si vous ne trouvez pas l'option correspondant à votre groupe, veuillez l'ajouter dans le champ dédié au
+      %span
+        = link_to 'routage', champs_admin_procedure_path(@procedure_id)
 
-  %ul.fr-btns-group.fr-btns-group--sm.fr-btns-group--inline.fr-btns-group--icon-right
-    %li
-      - if @groupe_instructeur.can_delete?
-        %p= t('.delete')
-        = button_to admin_procedure_groupe_instructeur_path(@procedure_id, @groupe_instructeur),
-        class: 'fr-btn fr-btn--tertiary fr-btn--icon-left fr-icon-delete-line',
-        method: :delete,
-        data: { confirm: t('.delete_confirmation', group_name: @groupe_instructeur.label) } do
-          Supprimer
-      - else
-        = button_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(@procedure_id, @groupe_instructeur),
-        class: 'fr-btn fr-btn--tertiary fr-icon-folder-2-line',
-        title: t('.move_files_confirmation'),
-        method: :get do
-          = t('.move_files', count: @groupe_instructeur.dossiers.visible_by_administration.size)
+  .flex.fr-btns-group--sm.fr-btns-group--inline.fr-btns-group--icon-right
+    - if @groupe_instructeur.can_delete?
+      %p= t('.delete')
+      = button_to admin_procedure_groupe_instructeur_path(@procedure_id, @groupe_instructeur),
+      class: 'fr-btn fr-btn--tertiary fr-btn--icon-left fr-icon-delete-line',
+      method: :delete,
+      data: { confirm: t('.delete_confirmation', group_name: @groupe_instructeur.label) } do
+        Supprimer
+    - else
+      = button_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(@procedure_id, @groupe_instructeur),
+      class: 'fr-btn fr-btn--tertiary fr-icon-folder-2-line',
+      title: t('.move_files_confirmation'),
+      method: :get do
+        = t('.move_files', count: @groupe_instructeur.dossiers.visible_by_administration.size)

spec/system/routing/rules_full_scenario_spec.rb

@@ -60,7 +60,7 @@ describe 'The routing with rules', js: true do
 
     # update defaut groupe
     click_on 'défaut'
-    expect(page).to have_text('Paramètres principaux')
+    expect(page).to have_text('Paramètres du groupe')
     fill_in 'Nom du groupe', with: 'littéraire'
     click_on 'Renommer'
     expect(page).to have_text('Le nom est à présent « littéraire ». ')