Project

General

Profile

Bug #71238

template menu et class .contains-hidden-elements

Added by Thomas Jund 3 months ago. Updated 3 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
10 November 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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`.

History

#1

Updated by Agate Berriot 3 months ago

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.)

Also available in: Atom PDF