entrouvert
/
gadjo
15
0
Fork 0
Code Issues Pull Requests 3 Releases Activity

Avoir les textes d'aide des champs dans des <p> (#84751) #49

Merged
fpeters merged 2 commits from wip/84751-rgaa-put-help-text-in-p into main 2024-04-16 10:06:17 +02:00
Conversation 1 Commits 2 Files Changed 2 +4 -1
nroche commented 2023-12-14 16:11:59 +01:00
Owner
Copy Link
No description provided.
nroche added 1 commit 2023-12-14 16:11:59 +01:00
gitea/gadjo/pipeline/head This commit looks good Details
85e0db67bc widgets: put field help texts in <p> (#84751)
nroche commented 2023-12-14 16:14:00 +01:00
Author
Owner
Copy Link

Vu le cartouche du ticket client, ça ne concerne que la page login.
Cependant je vais certainement toucher beaucoup plus large, et je n'arrive pas à trouver où c'est utilisé pour vérifier que je ne casse rien.

Voici les deux items identifiés comme problématiques corrigés par ce patch.

<span class="optional">(optionnel)</<span class="optional">(optionnel)</span>
et
<div class="hint" id="help_text_id_remember_me">Ne pas demander d’authentification au prochain accès</div>span>

Le premier n'est pas activé (_form.scss ligne 833):

/* mark forms optional fields and hide the required ones */
.widget-optional span.optional {
	display: none;
}

Pour le second, les marges sont déjà annulées (_forms.scss ligne 402)

span.helptext p, div.hint p {
    margin: 0;
}
Vu le cartouche du ticket client, ça ne concerne que la page login. Cependant je vais certainement toucher beaucoup plus large, et je n'arrive pas à trouver où c'est utilisé pour vérifier que je ne casse rien. Voici les deux items identifiés comme problématiques corrigés par ce patch. `<span class="optional">(optionnel)</<span class="optional">(optionnel)</span>` et `<div class="hint" id="help_text_id_remember_me">Ne pas demander d’authentification au prochain accès</div>span>` Le premier n'est pas activé (_form.scss ligne 833): ``` /* mark forms optional fields and hide the required ones */ .widget-optional span.optional { display: none; } ``` Pour le second, les marges sont déjà annulées (_forms.scss ligne 402) ``` span.helptext p, div.hint p { margin: 0; } ```
fpeters reviewed 2023-12-14 17:26:53 +01:00
gadjo/templates/gadjo/widget.html Outdated
@ -12,3 +12,3 @@
<span title="{% trans "This field is required." %}" class="required">*</span>
{% else %}
<span class="optional">{% trans "(optional)" %}</span>
<span class="optional"><p>{% trans "(optional)" %}<p></span>
fpeters commented 2023-12-14 17:26:53 +01:00
Owner
Copy Link

Tu ne peux pas avoir un <p> (type block) dans un <span> (type inline).

Tu ne peux pas avoir un `<p>` (type block) dans un `<span>` (type inline).
fpeters commented 2023-12-14 17:34:25 +01:00
Owner
Copy Link

Pour référence dans w.c.s. le balisage est de mettre le span de l'info optionnel/obligatoire à l'intérieur du <label>.

Pour référence dans w.c.s. le balisage est de mettre le span de l'info optionnel/obligatoire à l'intérieur du `<label>`.
nroche commented 2023-12-15 10:43:10 +01:00
Author
Owner
Copy Link

Oui, merci.

Oui, merci.
nroche marked this conversation as resolved
fpeters requested changes 2023-12-15 09:59:24 +01:00
Dismissed
fpeters left a comment
Owner
Copy Link

cf note plus haut.

cf note plus haut.
nroche force-pushed wip/84751-rgaa-put-help-text-in-p from 85e0db67bc to 10c6d30ec3 2023-12-15 10:39:28 +01:00 Compare
nroche changed title from Avoir les textes d'aide des champs dans des <p> (#84751) to Avoir les textes d'aide des champs dans des <p> et <label> (#84751) 2023-12-15 10:51:57 +01:00
nroche force-pushed wip/84751-rgaa-put-help-text-in-p from 10c6d30ec3 to b7f1a343be 2023-12-15 10:52:37 +01:00 Compare
fpeters requested changes 2023-12-15 11:04:11 +01:00
Dismissed
gadjo/templates/gadjo/widget.html Outdated
@ -12,3 +12,3 @@
<span title="{% trans "This field is required." %}" class="required">*</span>
{% else %}
<span class="optional">{% trans "(optional)" %}</span>
<span class="optional"><label>{% trans "(optional)" %}</label></span>
fpeters commented 2023-12-15 11:04:09 +01:00
Owner
Copy Link

Dans w.c.s. : à l'intérieur du label, le libellé du champ. Pas un label supplémentaire sans lien avec rien. (ceci ne va pas du tout passer côté accessibilité).

Dans w.c.s. : à l'intérieur du label, le libellé du champ. Pas un label supplémentaire sans lien avec rien. (ceci ne va pas du tout passer côté accessibilité).
nroche commented 2023-12-15 11:40:11 +01:00
Author
Owner
Copy Link

Pour référence dans w.c.s. le balisage est de mettre le span de l'info optionnel/obligatoire à l'intérieur du <label>.
ex:

<div class="title">
  <label class="field--label" id="form_label_f87042b7f-f8a7-4fac-b32a-dc195f3f9db2" for="form_f87042b7f-f8a7-4fac-b32a-dc195f3f9db2">
    créneau
    <span title="Ce champ est obligatoire." class="required">*</span>
  </label>
</div>

Or ici il est à l'extérieur.
J'imagine, parce que {{ field.label_tag }} ne prévoit rien pour l'intégrer.

<div class="title" id="id_remember_me_title">
  <label for="id_remember_me">Se souvenir de moi&nbsp;:</label>
  <span class="optional"><label>(optionnel)</label></span>
</div>

Je retire ce second point du patch, parce que :

Utiliser la propriété display avec la valeur none sur un élément entraînera son retrait de l'arbre d'accessibilité. Cet élément ainsi que ses descendants ne seront plus annoncés par les lecteurs d'écrans.

> Pour référence dans w.c.s. le balisage est de mettre le span de l'info optionnel/obligatoire à l'intérieur du `<label>`. ex: ``` <div class="title"> <label class="field--label" id="form_label_f87042b7f-f8a7-4fac-b32a-dc195f3f9db2" for="form_f87042b7f-f8a7-4fac-b32a-dc195f3f9db2"> créneau <span title="Ce champ est obligatoire." class="required">*</span> </label> </div> ``` Or ici il est à l'extérieur. J'imagine, parce que `{{ field.label_tag }}` ne prévoit rien pour l'intégrer. ``` <div class="title" id="id_remember_me_title"> <label for="id_remember_me">Se souvenir de moi&nbsp;:</label> <span class="optional"><label>(optionnel)</label></span> </div> ``` Je retire ce second point du patch, parce que : > Utiliser la propriété display avec la valeur none sur un élément entraînera son retrait de l'arbre d'accessibilité. Cet élément ainsi que ses descendants ne seront plus annoncés par les lecteurs d'écrans.
nroche marked this conversation as resolved
nroche force-pushed wip/84751-rgaa-put-help-text-in-p from b7f1a343be to 9df1243503 2023-12-15 11:37:50 +01:00 Compare
nroche changed title from Avoir les textes d'aide des champs dans des <p> et <label> (#84751) to Avoir les textes d'aide des champs dans des <p> (#84751) 2023-12-15 11:40:35 +01:00
fpeters added 1 commit 2023-12-22 10:01:17 +01:00
gitea/gadjo/pipeline/head This commit looks good Details
f9d92fffae scss: remove margin for help text paragraph (#84751)
fpeters reviewed 2023-12-22 10:03:09 +01:00
gadjo/templates/gadjo/widget.html
@ -22,3 +22,3 @@
{% block widget-hint %}
{% if field.help_text %}
<div class="hint" id="help_text_{{field.id_for_label}}">{{ field.help_text|safe }}</div>
<div class="hint" id="help_text_{{field.id_for_label}}"><p>{{ field.help_text|safe }}</p></div>
fpeters commented 2023-12-22 10:03:09 +01:00
Owner
Copy Link

Changer le balisage peut souvent amener un changement de rendu; le style pour les paragraphes est d'avoir des marges avant/après, ce qui ne va pas être souhaité ici. J'ai ajouté un commit. (équivalent à ce qui est fait dans publik-base-theme).

Changer le balisage peut souvent amener un changement de rendu; le style pour les paragraphes est d'avoir des marges avant/après, ce qui ne va pas être souhaité ici. J'ai ajouté un commit. (équivalent à ce qui est fait dans publik-base-theme).
nroche commented 2024-01-10 16:38:11 +01:00
Author
Owner
Copy Link

J'ai ajouté un commit.

Merci.
Je n'ai regardé que le rendu sur le ticket client (la page d'authentification) et donc j'ai zappé le rendu en backoffice
(en y regardant à nouveau, je n'ai pas trouvé où ça s'applique, mis à part sur la saisie w.c.s. où c'est déjà pris en charge, mais oui comme ça ce sera prévu).

> J'ai ajouté un commit. Merci. Je n'ai regardé que le rendu sur le ticket client (la page d'authentification) et donc j'ai zappé le rendu en backoffice (en y regardant à nouveau, je n'ai pas trouvé où ça s'applique, mis à part sur la saisie w.c.s. où c'est déjà pris en charge, mais oui comme ça ce sera prévu).
nroche marked this conversation as resolved
fpeters approved these changes 2024-04-16 10:06:11 +02:00
fpeters left a comment
Owner
Copy Link

Un peu oubliée, désolé, je valide et envoie.

Un peu oubliée, désolé, je valide et envoie.
fpeters merged commit 35aa41eec9 into main 2024-04-16 10:06:17 +02:00
fpeters deleted branch wip/84751-rgaa-put-help-text-in-p 2024-04-16 10:06:17 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/gadjo#49
No description provided.
Delete Branch "wip/84751-rgaa-put-help-text-in-p"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?