Prévient les listes vides

[LucienMLD]

🎯 Objectif

Dans le header il peut y avoir des <ul> vide s'il n'y a pas de contenue.

🔍 Implémentation

Mettre la liste à l'intérieur du bloc

⚠️ Informations supplémentaires

D'après ce que j'ai pu voir c'est au niveau du select du thème. Sur Wgtail si on choisit de ne pas afficher le select il reste une balise vide.

@Ash-Crow je n'ai pas en tête si ça peut avoir des incidences sur des projets qui utilisent Django DSFR.

🖼️ Images

[Ash-Crow]

Il faudrait introduire un bloc avec un autre nom je pense, pour ne pas casser les utilisations actuelles.

[LucienMLD]

Il faudrait introduire un bloc avec un autre nom je pense, pour ne pas casser les utilisations actuelles.

Si on crée un nouveau bloc on aura toujours une balise <ul> de vide non ?

[Ash-Crow]

L'idée est d'avoir à la fois le bloc externe et le bloc interne. Comme ça on ne casse pas les installations existantes, et celles qui en ont besoin peuvent mettre le bloc externe vide dans leur template pour que la liste ne soit pas générée (ça a été fait pour d'autres blocs aussi, cf. http://127.0.0.1:8765/django-dsfr/components/footer/#blocs-deprecies )

[Ash-Crow]

Il faudrait annuler les changements à ce fichier du coup :)

[Ash-Crow]

J'ai fait la modif (j'ai même ajouté deux blocs : un bloc header_tools_links autour de la liste de liens elle-même, et un header_tools_wrapper autour du groupe liens + recherche pour les sites qui n'utilisent ni l'un ni l'autre)

[LucienMLD]

Ok je viens de comprendre le raisonnement, c'est parfait