From f1838f37676fab63e44264319c6115d129357da7 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Mon, 29 Jul 2019 10:41:58 +0200 Subject: [PATCH 1/4] dossiers: fix edit button being visible on locked dossiers Fix #4143 --- app/views/users/dossiers/show/_header.html.haml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/views/users/dossiers/show/_header.html.haml b/app/views/users/dossiers/show/_header.html.haml index 8753abecb..910e220a7 100644 --- a/app/views/users/dossiers/show/_header.html.haml +++ b/app/views/users/dossiers/show/_header.html.haml @@ -11,7 +11,8 @@ = "- Déposé le #{l(dossier.en_construction_at, format: '%d %B %Y')}" - if current_user.owns?(dossier) - = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" + - if dossier.can_be_updated_by_user? + = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" = render partial: 'invites/dropdown', locals: { dossier: dossier } .clearfix From 7b345365fd9e4b92fbe6d076189d93783fedf1d8 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Mon, 29 Jul 2019 10:45:49 +0200 Subject: [PATCH 2/4] dossiers: fix the layout of the Edit button --- app/assets/stylesheets/new_design/dossier_show.scss | 4 ---- app/views/invites/_dropdown.html.haml | 2 +- app/views/users/dossiers/show/_header.html.haml | 8 ++++---- 3 files changed, 5 insertions(+), 9 deletions(-) diff --git a/app/assets/stylesheets/new_design/dossier_show.scss b/app/assets/stylesheets/new_design/dossier_show.scss index 971c7cc70..e26367a75 100644 --- a/app/assets/stylesheets/new_design/dossier_show.scss +++ b/app/assets/stylesheets/new_design/dossier_show.scss @@ -38,10 +38,6 @@ } } - .button.edit-form { - float: right; - } - .messagerie-explanation { margin-bottom: $default-padding * 2; } diff --git a/app/views/invites/_dropdown.html.haml b/app/views/invites/_dropdown.html.haml index 869804b53..368c359ef 100644 --- a/app/views/invites/_dropdown.html.haml +++ b/app/views/invites/_dropdown.html.haml @@ -1,4 +1,4 @@ -%span.dropdown.invite-user-action{ :style => "float:right;right: 10px;" } +%span.dropdown.invite-user-action %button.button.dropdown-button %span.icon.person - if dossier.invites.count > 0 diff --git a/app/views/users/dossiers/show/_header.html.haml b/app/views/users/dossiers/show/_header.html.haml index 910e220a7..96341a4fe 100644 --- a/app/views/users/dossiers/show/_header.html.haml +++ b/app/views/users/dossiers/show/_header.html.haml @@ -11,10 +11,10 @@ = "- Déposé le #{l(dossier.en_construction_at, format: '%d %B %Y')}" - if current_user.owns?(dossier) - - if dossier.can_be_updated_by_user? - = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" - = render partial: 'invites/dropdown', locals: { dossier: dossier } - .clearfix + .header-actions + = render partial: 'invites/dropdown', locals: { dossier: dossier } + - if dossier.can_be_updated_by_user? + = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" %ul.tabs = dynamic_tab_item('Résumé', dossier_path(dossier)) From e99edda7b19642f483cc2df02feed4af8741e388 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Mon, 29 Jul 2019 10:49:49 +0200 Subject: [PATCH 3/4] dossiers: hide the Edit button on the actual edit page Otherwise the "Edit" button may be confused for a "Save" button. --- app/views/users/dossiers/show/_header.html.haml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/views/users/dossiers/show/_header.html.haml b/app/views/users/dossiers/show/_header.html.haml index 96341a4fe..51d6a2797 100644 --- a/app/views/users/dossiers/show/_header.html.haml +++ b/app/views/users/dossiers/show/_header.html.haml @@ -13,8 +13,8 @@ - if current_user.owns?(dossier) .header-actions = render partial: 'invites/dropdown', locals: { dossier: dossier } - - if dossier.can_be_updated_by_user? - = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" + - if dossier.can_be_updated_by_user? && !current_page?(modifier_dossier_path(dossier)) + = link_to "Modifier mon dossier", modifier_dossier_path(dossier), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction" %ul.tabs = dynamic_tab_item('Résumé', dossier_path(dossier)) From bcd5182446079a32932e6241d98746ab25a10c3a Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Mon, 29 Jul 2019 10:54:19 +0200 Subject: [PATCH 4/4] dossier: hide the draft explanation when already submitted --- app/views/shared/dossiers/_edit.html.haml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/views/shared/dossiers/_edit.html.haml b/app/views/shared/dossiers/_edit.html.haml index ed34765e0..3a03f3abd 100644 --- a/app/views/shared/dossiers/_edit.html.haml +++ b/app/views/shared/dossiers/_edit.html.haml @@ -15,8 +15,9 @@ Les champs avec un astérisque ( %span.mandatory> * ) sont obligatoires. - %p.mandatory-explanation - Pour enregistrer votre dossier et le reprendre plus tard, cliquez sur le bouton « Enregistrer le brouillon » en bas à gauche du formulaire. + - if dossier.brouillon? + %p.mandatory-explanation + Pour enregistrer votre dossier et le reprendre plus tard, cliquez sur le bouton « Enregistrer le brouillon » en bas à gauche du formulaire. - if notice_url(dossier.procedure).present? = link_to notice_url(dossier.procedure), target: '_blank', rel: 'noopener', class: 'button notice', title: "Pour vous aider à remplir votre dossier, vous pouvez consulter le guide de cette démarche." do