From dd1b67bbc7a952d95efabcc5a1d24bf68301e7dd Mon Sep 17 00:00:00 2001 From: Eric Leroy-Terquem Date: Wed, 7 Jun 2023 11:00:38 +0200 Subject: [PATCH 1/3] UI(groupe instructeur): update heading level and wording --- .../one_groupe_management_component.html.haml | 2 +- spec/system/routing/rules_full_scenario_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml b/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml index 67231a72d..fbb3004d4 100644 --- a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml +++ b/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), diff --git a/spec/system/routing/rules_full_scenario_spec.rb b/spec/system/routing/rules_full_scenario_spec.rb index 93b487e10..94a1b9082 100644 --- a/spec/system/routing/rules_full_scenario_spec.rb +++ b/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 ». ') From f2dffc2ba3f46d180dd361304ac8755063eb0099 Mon Sep 17 00:00:00 2001 From: Eric Leroy-Terquem Date: Wed, 7 Jun 2023 11:01:28 +0200 Subject: [PATCH 2/3] UI(groupe instructeur): add a hint for adding routing value --- .../one_groupe_management_component.html.haml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml b/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml index fbb3004d4..9c2668a8f 100644 --- a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml +++ b/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml @@ -31,13 +31,17 @@ - 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 From 68c81c8d230fe3fe509c929e81930df591c44e50 Mon Sep 17 00:00:00 2001 From: Eric Leroy-Terquem Date: Wed, 7 Jun 2023 11:49:22 +0200 Subject: [PATCH 3/3] UI(groupe instructeur): update css to keep content in container --- .../one_groupe_management_component.html.haml | 29 +++++++++---------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml b/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml index 9c2668a8f..8969290e7 100644 --- a/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml +++ b/app/components/procedure/one_groupe_management_component/one_groupe_management_component.html.haml @@ -43,18 +43,17 @@ %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)