Project

General

Profile

Bug #71238

template menu et class .contains-hidden-elements

Added by Thomas Jund about 2 years ago. Updated about 2 years 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`.


Related issues

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

Actions

History

#1

Updated by A. B. about 2 years 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.)

#2

Updated by Thomas Jund 11 months ago

  • Related to Bug #86069: menu : classe "contains-hidden-elements" distribuée trop généreusement aux entrées du menu added

Also available in: Atom PDF