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
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
gadjo/static/css/_forms.scss
View File

@ -292,6 +292,9 @@ span.helptext,
div.hint {
display: block;
font-size: 80%;
p {
margin: 0;
}
}
form p label {

2
gadjo/templates/gadjo/widget.html
View File

@ -21,7 +21,7 @@
{% block widget-attrs %}{% endblock %}>
{% 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>
nroche marked this conversation as resolved
fpeters commented 2023-12-22 10:03:09 +01:00
Review
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
Review
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).
{% endif %}
{% endblock %}
{% block widget-control %}