Merge pull request #8666 from colinux/fix-personnes-impliquees-combobox
fix(instructeurs): combobox layout in "personnes impliquees" tab
This commit is contained in:
commit
e87bb24872
3 changed files with 12 additions and 13 deletions
|
@ -10,7 +10,7 @@
|
|||
}
|
||||
|
||||
[data-react-component-value^="ComboMultiple"] {
|
||||
margin-bottom: $default-fields-spacer;
|
||||
margin-bottom: 0;
|
||||
|
||||
[data-reach-combobox-token-list] {
|
||||
padding: 0.5 * $default-padding;
|
||||
|
|
|
@ -6,14 +6,13 @@
|
|||
- else
|
||||
%p.tab-paragrah.mb-1
|
||||
Le destinataire suivra automatiquement le dossier
|
||||
= form_for dossier, url: send_to_instructeurs_instructeur_dossier_path(dossier.procedure, dossier), method: :post, html: { class: 'form recipients-form' } do |f|
|
||||
.flex.justify-start.align-start
|
||||
= hidden_field_tag :recipients, nil
|
||||
= react_component("ComboMultiple",
|
||||
options: potential_recipients.map{|r| [r.email, r.id]},
|
||||
selected: [], disabled: [],
|
||||
group: '.recipients-form',
|
||||
name: 'recipients',
|
||||
label: 'Emails')
|
||||
= form_for dossier, url: send_to_instructeurs_instructeur_dossier_path(dossier.procedure, dossier), method: :post, html: { class: 'form recipients-form fr-mb-4w' } do |f|
|
||||
= hidden_field_tag :recipients, nil
|
||||
= react_component("ComboMultiple",
|
||||
options: potential_recipients.map{|r| [r.email, r.id]},
|
||||
selected: [], disabled: [],
|
||||
group: '.recipients-form',
|
||||
name: 'recipients',
|
||||
label: 'Emails')
|
||||
|
||||
= f.submit "Envoyer", class: "button large send gap-left"
|
||||
= f.submit "Envoyer", class: "fr-btn fr-mt-2w"
|
||||
|
|
|
@ -14,14 +14,14 @@ describe 'instructeurs/dossiers/envoyer_dossier_block.html.haml', type: :view do
|
|||
let(:potential_recipients) { [instructeur] }
|
||||
|
||||
it { is_expected.to match(/data-react-props.*#{instructeur.email}/) }
|
||||
it { is_expected.to have_css(".button.send") }
|
||||
it { is_expected.to have_css(".fr-btn") }
|
||||
end
|
||||
|
||||
context "there is no other instructeur for the procedure" do
|
||||
let(:potential_recipients) { [] }
|
||||
|
||||
it { is_expected.not_to have_css("select") }
|
||||
it { is_expected.not_to have_css(".button.send") }
|
||||
it { is_expected.not_to have_css(".fr-btn") }
|
||||
it { is_expected.to have_content("Vous êtes le seul instructeur assigné sur cette démarche") }
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue