Development #45896
définir une classe pour marquer un formulaire où les astérisques ne doivent pas être ajoutées aux champs obligatoires
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Lié à Development #40159: templates: utilise le template tag with_template pour le rendu des formulaires ajouté
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Fichier 0001-forms-add-class-to-mark-form-s-optional-fields-45896.patch 0001-forms-add-class-to-mark-form-s-optional-fields-45896.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 3 ans
Je me demande s'il ne faudrait pas plutôt faire appel à sr-only pour les champs obligatoires, qu'ils soient toujours explicités obligatoires aux lecteurs d'écran.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Je pense que les lecteurs d'écran supportent assez bien les attributs `required` et `aria-required="true"`.
(J'ai noté qu'il y avait des widgets (comme les radios groupes) qui n'avaient pas d'attribut indiquant l'aspect obligatoire).
La class est donc destinée à visuellement mettre en évidence les champs d'un formulaire optionnels plutôt que requis.
(Je n'ai toujours pas compris l'intérêt ou le cas d'usage, mais passons)
Pourquoi systématiquement chercher à alourdir le poids des sélecteurs ?
.pk-mark-optional-fields {
span.required {
display: none;
}
span.optional {
display: inline;
font-style: italic;
}
}
Ça ne suffit pas ?
Ensuite, un petit commentaire au dessus de la class ne serait pas de refus :).
Mis à jour par Frédéric Péters il y a plus de 3 ans
Je pense que les lecteurs d'écran supportent assez bien les attributs `required` et `aria-required="true"`.
Modulo les audits qui parfois écrivent que ça doit se combiner avec du texte explicite.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Modulo les audits qui parfois écrivent que ça doit se combiner avec du texte explicite.
Dans ce cas, utiliser .sr-only serait une bonne solution.
et aussi corriger la sémantique du tag required en utilisant un abbr plutôt qu'un span
<abbr title="Ce champ est obligatoire." class="required">*</abbr>
Mis à jour par Frédéric Péters il y a plus de 3 ans
et aussi corriger la sémantique du tag required en utilisant un abbr plutôt qu'un span
(à ne pas considérer dans ce ticket, la balise n'étant pas dans publik-base-theme, et parce que ce changement demanderait de toucher au moins gadjo et wcs et les intégrations graphiques auquotidien).
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Fichier 0001-forms-add-class-to-mark-form-s-optional-fields-45896.patch 0001-forms-add-class-to-mark-form-s-optional-fields-45896.patch ajouté
- Assigné à mis à Serghei Mihai
En utilisant sr-only.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Soupir sur le patch authentic poussé avant validation de celui-ci.
Je n'ai pas compris pourquoi ici les "optionnels" n'étaient pas cachés par défaut (contrairement à gadjo).
J'ai tapé en last minute ça :
@@ -623,6 +623,10 @@
/* mark forms optional fields and hide the required ones */
+.widget-optional span.optional {
+ display: none;
+}
+
.pk-mark-optional-fields {
.widget-required {
span.required {
@@ -630,6 +634,7 @@ div.select2-search {
}
}
.widget-optional span.optional {
+ display: inline;
font-style: italic;
}
}
Et j'ai poussé.
commit b039847a92c097843c9f1ac8e526477198b81843
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Fri Aug 14 12:18:01 2020 +0200
forms: add class to mark form's optional fields (#45896)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: add class to mark form's optional fields (#45896)