Bug: La balise p de la description du footer ne devrait pas être présente si elle est vide

[Ash-Crow]

Erreur 8.9.1 du RGAA

[cyrilf]

Hey, je suis tombé là un peu par hasard.

J'ai l'impression qu'il faudrait faire des changements dans le repo django-dsfr.

• Changer le template footer.html https://github.com/numerique-gouv/django-dsfr/blob/main/dsfr/templates/dsfr/footer.html#L15-L19 :

• {% block footer_content %}

• {{ SITE_CONFIG.footer_description | safe }}

• {% endblock footer_content %}

pour que la balise p soit incluse dans le block footer_content et qu'on puisse la sur-charger. Par contre, ça ressemble un peu à un breaking change.. :unamused:

• Mettre à jour/publier une nouvelle version

---

Modifications dans ce repo :

• Mettre à jour la dépendance vers la nouvelle version django-dsfr
• Mettre à jour le footer : https://github.com/numerique-gouv/content-manager/blob/main/templates/blocks/footer.html

  
  {% block footer_content %}

• {% if settings.content_manager.CmsDsfrConfig.footer_description %}

• {{ settings.content_manager.CmsDsfrConfig.footer_description | safe }}

• {% endif %} {% endblock footer_content %}

---

Je n'ai pas pris le temps de faire une PR parce que je suis juste de passage sur ce repo :wink: Il y a surement une meilleure approche qui n'implique pas de modifier django-dsfr.

[Ash-Crow]

Je pense qu'il vaut mieux procéder par étapes, avec un nouveau bloc footer_description qui englobe l'autre et la balise pour donner le temps aux utilisateurs de mettre à jour sans casser immédiatement.