Merge pull request #6765 from tchak/fix-a11y

Fix a11y

app/assets/stylesheets/custom_reset.scss

@@ -4,6 +4,7 @@
 html,
 body {
   height: 100%;
+  background-color: $white;
 }
 
 html {

app/assets/stylesheets/forms.scss

@@ -565,3 +565,8 @@
     margin-bottom: 16px;
   }
 }
+
+input::placeholder,
+textarea::placeholder {
+  color: $dark-grey;
+}

app/assets/stylesheets/table.scss

@@ -14,15 +14,16 @@
     vertical-align: middle;
   }
 
-  th {
+  th,
+  td.libelle {
     text-align: left;
     font-weight: bold;
     padding: (3 * $default-spacer) 2px;
-  }
 
-  th.padded {
+    &.padded {
       padding-left: (2 * $default-spacer);
     }
+  }
 
   &.hoverable {
     tbody tr:hover {
@@ -38,7 +39,8 @@
       border-top: none;
     }
 
-    th {
+    th,
+    td.libelle {
       @include vertical-padding($default-spacer);
 
       &.header-section {

app/views/layouts/_search_dossiers_form.html.haml

@@ -1,6 +1,5 @@
 .header-search{ role: 'search' }
   = form_tag "#{search_endpoint}", method: :get, class: "form" do
-    = label_tag :q, "Numéro de dossier", class: 'hidden'
+    = text_field_tag "q", "#{@search_terms if @search_terms.present?}", placeholder: "Rechercher un dossier", title: "Rechercher un dossier"
-    = text_field_tag "q", "#{@search_terms if @search_terms.present?}", placeholder: "Rechercher un dossier"
     %button{ title: "Rechercher" }
       = image_tag "icons/search-blue.svg", alt: 'Rechercher', 'aria-hidden':'true', width: 24, height: 24, loading: 'lazy'

app/views/shared/dossiers/_champ_row.html.haml

@@ -1,19 +1,19 @@
 - champs.reject(&:exclude_from_view?).each do |c|
   - if c.type_champ == TypeDeChamp.type_champs.fetch(:repetition)
     %tr
-      %th.libelle.repetition{ colspan: 3 }
+      %td.libelle.repetition{ colspan: 3 }
         = "#{c.libelle} :"
     - c.rows.each do |champs|
       = render partial: "shared/dossiers/champ_row", locals: { champs: champs, demande_seen_at: demande_seen_at, profile: profile, repetition: true }
       %tr
-        %th{ colspan: 4 }
+        %td.libelle{ colspan: 4 }
   - else
     %tr
       - if c.type_champ == TypeDeChamp.type_champs.fetch(:header_section)
         %th.header-section{ colspan: 3 }
           = c.libelle
       - else
-        %th.libelle{ class: repetition ? 'padded' : '' }
+        %td.libelle{ class: repetition ? 'padded' : '' }
           = "#{c.libelle} :"
         %td.rich-text
           %div{ class: highlight_if_unseen_class(demande_seen_at, c.updated_at) }

app/views/shared/dossiers/_champs.html.haml

@@ -1,7 +1,7 @@
 %table.table.vertical.dossier-champs
   %tbody
     - if dossier.show_groupe_instructeur_details?
-      %th= dossier.procedure.routing_criteria_name
+      %td.libelle= dossier.procedure.routing_criteria_name
       %td{ class: highlight_if_unseen_class(demande_seen_at, dossier.groupe_instructeur_updated_at) }= dossier.groupe_instructeur.label
       %td.updated-at
         %span{ class: highlight_if_unseen_class(demande_seen_at, dossier.groupe_instructeur_updated_at) }

app/views/shared/dossiers/_identite_entreprise.html.haml

@@ -5,59 +5,59 @@
         %td= t('warning_for_private_info', scope: 'views.shared.dossiers.identite_entreprise', etablissement: raison_sociale_or_name(etablissement))
     - else
       %tr
-        %th.libelle Dénomination :
+        %td.libelle Dénomination :
         %td= raison_sociale_or_name(etablissement)
       %tr
-        %th.libelle SIRET :
+        %td.libelle SIRET :
         %td= etablissement.siret
 
       - if etablissement.siret != etablissement.entreprise.siret_siege_social
         %tr
-          %th.libelle SIRET du siège social:
+          %td.libelle SIRET du siège social:
           %td= etablissement.entreprise.siret_siege_social
       %tr
-        %th.libelle Forme juridique :
+        %td.libelle Forme juridique :
         %td= sanitize(etablissement.entreprise.forme_juridique)
       %tr
-        %th.libelle Libellé NAF :
+        %td.libelle Libellé NAF :
         %td= etablissement.libelle_naf
       %tr
-        %th.libelle Code NAF :
+        %td.libelle Code NAF :
         %td= etablissement.naf
       %tr
-        %th.libelle Date de création :
+        %td.libelle Date de création :
         %td= try_format_date(etablissement.entreprise.date_creation)
       - if profile == 'instructeur'
         %tr
-          %th.libelle
+          %td.libelle
             Effectif mensuel
             = try_format_mois_effectif(etablissement)
             (URSSAF) :
           %td= etablissement.entreprise_effectif_mensuel
         %tr
-          %th.libelle
+          %td.libelle
             Effectif moyen annuel
             = etablissement.entreprise_effectif_annuel_annee
             (URSSAF) :
           %td= etablissement.entreprise_effectif_annuel
       %tr
-        %th.libelle Effectif de l'organisation (INSEE) :
+        %td.libelle Effectif de l'organisation (INSEE) :
         %td
           = effectif(etablissement)
       %tr
-        %th.libelle Numéro de TVA intracommunautaire :
+        %td.libelle Numéro de TVA intracommunautaire :
         %td= etablissement.entreprise.numero_tva_intracommunautaire
       %tr
-        %th.libelle Adresse :
+        %td.libelle Adresse :
         %td
           - etablissement.adresse.split("\n").each do |line|
             = line
             %br
       %tr
-        %th.libelle Capital social :
+        %td.libelle Capital social :
         %td= pretty_currency(etablissement.entreprise.capital_social)
       %tr
-        %th.libelle Chiffre d’affaires :
+        %td.libelle Chiffre d’affaires :
         %td
           - if profile == 'instructeur'
             %details
@@ -83,7 +83,7 @@
           = render partial: 'shared/dossiers/identite_entreprise_bilan_detail',
             locals: { libelle: 'Besoin en fonds de roulement', key: 'besoin_en_fonds_de_roulement', etablissement: etablissement }
           %tr
-            %th.libelle
+            %td.libelle
               Chiffres financiers clés (Banque de France)
               = "en #{pretty_currency_unit(etablissement.entreprise_bilans_bdf_monnaie)} :"
 
@@ -105,12 +105,12 @@
                 = link_to "au format ods", bilans_bdf_instructeur_dossier_path(procedure_id: @dossier.procedure.id, dossier_id: @dossier.id, format: 'ods')
         - else
           %tr
-            %th.libelle
+            %td.libelle
               Bilans Banque de France :
             %td Les 3 derniers bilans connus de votre entreprise par la Banque de France ont été joints à votre dossier.
       - if etablissement.entreprise_attestation_sociale.attached?
         %tr
-          %th.libelle Attestation sociale :
+          %td.libelle Attestation sociale :
           - if profile == 'instructeur'
             %td= link_to "Consulter l'attestation", url_for(etablissement.entreprise_attestation_sociale)
           - else
@@ -118,7 +118,7 @@
 
       - if etablissement.entreprise_attestation_fiscale.attached?
         %tr
-          %th.libelle Attestation fiscale :
+          %td.libelle Attestation fiscale :
           - if profile == 'instructeur'
             %td= link_to "Consulter l'attestation", url_for(etablissement.entreprise_attestation_fiscale)
           - else
@@ -126,22 +126,22 @@
 
       - if etablissement.association?
         %tr
-          %th.libelle Numéro RNA :
+          %td.libelle Numéro RNA :
           %td= etablissement.association_rna
         %tr
-          %th.libelle Titre :
+          %td.libelle Titre :
           %td= etablissement.association_titre
         %tr
-          %th.libelle Objet :
+          %td.libelle Objet :
           %td= etablissement.association_objet
         %tr
-          %th.libelle Date de création :
+          %td.libelle Date de création :
           %td= try_format_date(etablissement.association_date_creation)
         %tr
-          %th.libelle Date de publication :
+          %td.libelle Date de publication :
           %td= try_format_date(etablissement.association_date_publication)
         %tr
-          %th.libelle Date de déclaration :
+          %td.libelle Date de déclaration :
           %td= try_format_date(etablissement.association_date_declaration)
 
 %p

app/views/shared/dossiers/_identite_entreprise_bilan_detail.html.haml

@@ -1,5 +1,5 @@
 %tr
-  %th.libelle
+  %td.libelle
     = "#{libelle} :"
   %td
     %details

app/views/shared/dossiers/_identite_individual.html.haml

@@ -1,15 +1,15 @@
 %table.table.vertical.dossier-champs
   %tbody
     %tr
-      %th.libelle Civilité :
+      %td.libelle Civilité :
       %td= individual.gender
     %tr
-      %th.libelle Prénom :
+      %td.libelle Prénom :
       %td= individual.prenom
     %tr
-      %th.libelle Nom :
+      %td.libelle Nom :
       %td= individual.nom
     - if individual.birthdate.present?
       %tr
-        %th.libelle Date de naissance :
+        %td.libelle Date de naissance :
         %td= try_format_date(individual.birthdate)

app/views/shared/dossiers/_infos_generales.html.haml

@@ -1,11 +1,11 @@
 %table.table.vertical.dossier-champs
   %tbody
     %tr
-      %th.libelle Déposé le :
+      %td.libelle Déposé le :
       %td= l(dossier.depose_at, format: '%d %B %Y')
   - if dossier.justificatif_motivation.attached?
     %tr
-      %th.libelle Justificatif :
+      %td.libelle Justificatif :
       %td
         .action
           = render partial: 'shared/attachment/show', locals: { attachment: dossier.justificatif_motivation.attachment }

app/views/shared/dossiers/_user_infos.html.haml

@@ -1,5 +1,5 @@
 %table.table.vertical.dossier-champs
   %tbody
     %tr
-      %th.libelle Email :
+      %td.libelle Email :
       %td= user_deleted ? "#{email} (l’usager a supprimé son compte)" : email

app/views/shared/dossiers/messages/_form.html.haml

@@ -3,7 +3,7 @@
   - placeholder = t('views.shared.dossiers.messages.form.write_message_to_administration_placeholder')
   - if instructeur_signed_in? || administrateur_signed_in?
     - placeholder = t('views.shared.dossiers.messages.form.write_message_placeholder')
-  = f.text_area :body, rows: 5, placeholder: placeholder, required: true, class: 'message-textarea persisted-input'
+  = f.text_area :body, rows: 5, placeholder: placeholder, title: placeholder, required: true, class: 'message-textarea persisted-input'
   .flex.justify-between.wrap
     - disable_piece_jointe = defined?(disable_piece_jointe) ? disable_piece_jointe : false
     %div