Feat/6633 test que le html qu'on génère est toujours valide by artragis · Pull Request #6775 · zestedesavoir/zds-site

artragis · 2025-12-27T21:24:01Z

J'ai configuré un tool qui est assez commun en python pour faire de la validation.
Ensuite j'ai commencé à faire les petits fix qui fallait.

QA:

Attendre que la CI passe, je pense.

coveralls · 2026-01-23T21:08:31Z

coverage: 89.49% (+0.01%) from 89.476% — artragis:feat/6633_test_html into zestedesavoir:dev

Arnaud-D

Je n'ai pas tout testé en détail (il y a des changements qui nécessiteraient de regarder le fonctionnel attentivement), mais j'ai vu quelques soucis.

Arnaud-D · 2026-02-28T11:02:40Z

-            {% captureas alert_link %}{{ edit_link }}{% endcaptureas %}
+            {% captureas hide_link %}{{ edit_link|trim }}{% endcaptureas %}
+            {% captureas show_link %}{{ edit_link|trim }}{% endcaptureas %}
+            {% captureas alert_link %}{{ edit_link|trim }}{% endcaptureas %}


Je pense que trim n'est pas la meilleure solution pour le problème avec des retours à la ligne ou autres espaces dans les URL.

Dans le cas présent (et d'autres endroits du code), le problème, c'est qu'on utilise captureas pour avoir des variables dans le template qu'on devrait juste avoir avec une méthode dans le code. Ici, ça devrait être quelque chose du type message.url_of_edit_link. Cette méthode retournerait quelque chose de propre et nous éviterait de programmer dans le template pour rien.

Arnaud-D · 2026-02-28T11:03:25Z

                    {% include "misc/member_item.part.html" with member=authors.first author=True avatar=True deletable=False %}
                </li>
-            {% else %}
+            {% elif authors %}


J'ai l'impression que ça ne change rien par rapport à avant, parce que quand authors est vide, il ne s'affiche rien. Ou alors c'est une question de style ?

Arnaud-D · 2026-02-28T11:09:13Z

            "member_login_url": reverse("member-login")
            + "?next="
-            + quote(full_path)
+            + quote_plus(full_path)


Quitte à toucher ce genre de choses, peut-être utiliser l'argument query de reverse ? Ça permet de faire des URL avec des paramètres. Détails dans la doc.

Arnaud-D · 2026-02-28T11:10:59Z

            "member_login_url": reverse("member-login")
            + "?next="
-            + quote(full_path)
+            + quote_plus(full_path)


Quitte à toucher ce genre de cas, peut-être utiliser l'argument query de reverse ? Ça permet de faire des URL avec des paramètres. Détails dans la doc.

je connaissais pas bien vu

Arnaud-D · 2026-02-28T11:12:29Z

+        msg = _('<p>Pas assez de place ? <a href="{}?title={}&amp;{}">Envoyez un MP {}</a> !').format(
+            reverse("mp:create"), quote_plus(pm_title), usernames, plural


Un autre cas pour le paramètre query de reverse, je pense.

Arnaud-D · 2026-02-28T11:25:02Z

+    def add_extract_url(self):
+        root_content = self.top_container()
+        if self.parent.is_top_container:
+            return reverse("content:create-extract", args=[root_content.pk, root_content.slug, self.slug])
+        return reverse("content:create-extract", args=[root_content.pk, root_content.slug, self.parent.slug, self.slug])
+
+    @property
+    def add_container_url(self):


Pas super fan du nommage ici, parce que ça peut être lu comme "add_(extract_url)" alor que c'est "(add_extract)_url". L'anglais ne nous aide pas ici, mais peut-être qu'il faudrait être un peu plus verbeux avec un truc du type "url_to_add_extract" ou quelque chose du genre. Les mots de liaison ne sont pas interdits en anglais. :D

Arnaud-D · 2026-02-28T11:26:08Z

+        return response
+
+
+class ZdsTestCase(TestCase):


Est-ce que cette validation rallonge la suite de tests significativement ? Est-ce que tu as observé un changement à ce niveau-là ou non ?

Arnaud-D · 2026-02-28T11:32:52Z

                <div class="picto">
                    <div class="picto-hexagon">
-                        <img src="/static/images/logo-library.png" />
+                        <img src="/static/images/logo-library.png" alt="Logo de la bibliothèque ZDS"/>


Pour des images purement décoratives, je crois que la recommandation pour l'accessibilité est alt="" (et éventuellement un rôle en plus). J'ai trouvé un doc W3C qui en parle.

Tu peux confirmer que c'est toujours une bonne recommandation @AmauryCarrade ?

Arnaud-D · 2026-02-28T11:39:32Z

+<div class="content-header-container">
+    <h2 id="{{ child.position_in_parent }}-{{ child.slug }}"
+    {% if not child.is_validable %}
+        class="not-ready title-with-buttons"
+    {% else %}
+        class="title-with-buttons"
+    {% endif %}>
+        <a href="{{ base_url }}{{ child.get_url_path }}">{{ child.title }}</a>
+    </h2>
    {% if display_config.draft_actions.enable_edit %}
        <div class="actions-title">
-            <a href="{{ child.get_edit_url }}" class="ico-after edit btn btn-grey">
-                {% trans "Éditer" %}
-            </a>
-            <a href="#move-{{ child.slug }}" class="open-modal ico-after move btn btn-grey">{% trans "Déplacer" %}</a>
+            <ul>
+                <li>
+                <a href="{{ child.get_edit_url }}" class="ico-after edit btn btn-grey">{% trans "Éditer" %}</a>
+                </li>
+                <li>
+                <a href="#move-{{ child.slug }}" class="open-modal ico-after move btn btn-grey">{% trans "Déplacer" %}</a>
+                </li>
+                <li>
+                    {% include "tutorialv2/includes/sidebar/delete.part.html" with object=child additional_classes="ico-after cross btn btn-grey" %}
+                </li>
+            </ul>


Je ne sais pas si c'est cette modif précisément ou une autre, mais ça casse le design :

Arnaud-D · 2026-02-28T11:42:13Z

+        <h2 class="title-with-buttons">
            {% url "content:create-extract" content.pk content.slug as create_extract_url %}
            <a class="force-blue" href="{{ create_extract_url }}">
                {% trans "Ajouter une section" %}
            </a>
-            <div class="actions-title">
+            <span class="actions-title">
                <a href="{{ create_extract_url }}" class="ico-after more btn btn-grey">
                    {% trans "Ajouter" %}
                </a>
-            </div>
+            </span>


Design cassé ici également. Le bouton est devenu très long.

# Conflicts: # .github/workflows/ci.yml # zds/tutorialv2/tests/tests_views/tests_addcontributor.py

Situphen self-requested a review December 28, 2025 18:17

philippemilink added this to Suivi des PR Dec 28, 2025

github-project-automation Bot moved this to En développement in Suivi des PR Dec 28, 2025

Situphen removed their request for review December 29, 2025 11:59

Arnaud-D requested changes Feb 28, 2026

View reviewed changes

github-project-automation Bot moved this from En développement to Modification demandée in Suivi des PR Feb 28, 2026

artragis added 23 commits July 14, 2026 19:04

Automatically check HTML is valid

ddb2fcf

# Conflicts: # .github/workflows/ci.yml # zds/tutorialv2/tests/tests_views/tests_addcontributor.py

Fix invalid html in some forum pages

9eaf6b8

Properly use django templatetags to avoid & in url

7f1c2b6

Use a valid button type

ae909ce

Handle streaming response and follow param

fef0d8f

Improve html of content creation links

ad73c70

Ignore table attributes that are generated by ZMD in HTML

d0f05ab

Fix some attributes in galleries

abfd76e

Fix some attributes in content

ad0a771

remove div in h2 for edit/move/delete

505ee06

remove div in h2 for edit/move/delete

a40cdd8

Improve HTML & CSS for content, use JSONREsponse when necessary

f5a228b

Centralize bot creation to avoid empty profiles

a79540b

Use auto-id to avoid id duplication

2cf47c4

Fix image factory bug

418853f

add trim filter to avoid white spaces and new lines in urls

caf9193

Better css for tables

bf76852

use profile for external user

cd332b8

Fix test factory

be1027b

Ignore some canvas elements

623b7c0

Fix test

a412ce8

Fix test

9d4706b

Fix test

4bc45df

artragis added 9 commits July 14, 2026 19:07

do not show solved_by if topic is not solved

46b23eb

Meilleur html dans featured_resource_item.part.html

a1913d4

Bunch of HTML fixes

7f2bd26

Fix JsonResponse use

30dc4ac

Fix test

e5af767

Fix test

6aea815

Fix mp html

1ce6f90

Fix auto_id and missing href

5421a8b

Fix auto_id

8a360e5

artragis force-pushed the feat/6633_test_html branch from bf08987 to 8a360e5 Compare July 14, 2026 17:08

		msg = _('<p>Pas assez de place ? <a href="{}?title={}&{}">Envoyez un MP {}</a> !').format(
		reverse("mp:create"), quote_plus(pm_title), usernames, plural

Uh oh!

Conversation

artragis commented Dec 27, 2025

QA:

Uh oh!

coveralls commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Arnaud-D left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

coveralls commented Jan 23, 2026 •

edited

Loading

Arnaud-D left a comment •

edited

Loading