Projet

Général

Profil

Bug #71238

template menu et class .contains-hidden-elements

Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
10 novembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Le template menu pose une class `.contains-hidden-elements` sur les items de premier niveau.

class="{% if depth > 1 %}contains-hidden-elements{% endif %}

Mais comme le template nav.html de PBT, déclare `depth=2`

{% show_menu depth=2 ignore_visibility=True %}

Le class est systématiquement présente, qu'il y a une sous-nav ou pas. Je ne vois pas l'intérêt.
Ensuite, la sémantique de cette class laisse dire que l'item de nav possède des éléments masqués. Ce qui est faux, même dans le cas de la présence d'un sous-menu, à la fois dans le contexte d'une cellule menu comme celui de la nav principale en mobile.

Si l'objectif de cette class est d'indiquer si l'item de menu possède un sous-niveau (et j'en aurais besoin pour le thème du CD06), je propose que la class ajouté sois simplement `has-submenu`.


Demandes liées

Lié à Combo - Bug #86069: menu : classe "contains-hidden-elements" distribuée trop généreusement aux entrées du menuFermé24 janvier 2024

Actions

Historique

#1

Mis à jour par A. Berriot il y a plus d'un an

J'avais rajouté cette classe pour #40930 et pouvoir gérer l'affichage / masquage du menu via un appui sur la touche echap (https://gitea.entrouvert.org/entrouvert/combo/commit/202c614849f676001df124f320c1c501288eecd8).

Ça me paraissait important de ne pas me baser sur la la classe has-submenu pour pouvoir réutiliser cette logique sur d'autres éléments (modales, tooltips, etc.)

#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 3 mois

  • Lié à Bug #86069: menu : classe "contains-hidden-elements" distribuée trop généreusement aux entrées du menu ajouté

Formats disponibles : Atom PDF