Commit Graph

11 Commits (master)

Author SHA1 Message Date
rastapopoulos@spip.org e02fad531a Gros bug, bizarre que pas rapporté plus tôt : on ne doit surtout pas commencer à afficher le sous-menu si l'entrée parente n'a rien sorti ! (par exemple une rubrique non publiée, etc) 4 years ago
tcharlss@bravecassine.com d8c69ae9c5 data-depth également pour la racine 5 years ago
tcharlss@bravecassine.com 837f26739e On ajoute des classes BEM aux menus et à leurs entrées pour se simplifier la vie (les anciennes classes sont conservées bien sûr).
Ce qui donne le markup suivant :

ul.menu-items
  li.menu-items__item
    a.menu-items__lien

Ajout également d'un indicateur de la profondeur des sous-menus : data-depth="N".
Ainsi plus besoin de se casser la tête lorsqu'on veut cibler un sous-menu à une profondeur particulière, par exemple pour un sous-menu de profondeur 2 :

Avant : .menu-items > .item > .menu-items > .item > .menu-items
Après : menu-items[data-depth="2"]

Il y a juste l'entrée de menu « liste ou arborescence de rubriques et d'articles » pour laquelle la profondeur n'est pas indiquée correctement, là tout de suite je n'ai pas le courage de me plonger dans ce labyrinthe lovecraftien.
5 years ago
rastapopoulos@spip.org 0180280d47 Correction d'un problème quand on veut utiliser des listes en inline : les nettoyages de kent1 avaient collés tous les <li> ensemble, et donc pour les gens qui faisaient des listes inline ça collait tous les mots ensemble aussi, au lieu d'avoir un espace, donc ça pétait l'affichage et la lecture. On remet juste un retour à la ligne. 6 years ago
kent1@arscenic.info ed640ed13e #FILTRE{trim} pour éviter trop de lignes blanches laissées en route 7 years ago
cedric@yterium.com 34ca1f6bbb r81761 est une erreur : on a besoin de la classe sur les sous items aussi dans certains cas. On la pose donc partout, et on l'enlevera en JS si on ne la veut pas 10 years ago
cedric@yterium.com abaaa15a5d Attention : on ne veut poser la class extra que sur le premier niveau du menu. On passe donc un argument niveau que l'on incremente (compatibilite BootStrap ou autre) 10 years ago
cedric@yterium.com 5dc136a6e0 Ajouter la classe "nav" systematiquement sur tous les menus provoque des perturbations de la force (et du plugin menu Babbibel notamment) - nat33
On remplace donc par un #ENV{class} a toute fin utile et l'insertion de la constante _MENUS_EXTRA_CLASS (par defaut non definie, donc on n'insere aucune classe) et qui sera definie par les framework qui ont besoin d'une classe specifique a cet endroit (plugin Bootstrap par exemple) ou dans un mes_options.php avec par exemple :
define('_MENUS_EXTRA_CLASS','nav');
10 years ago
cedric@yterium.com d32481479e Gros fail perf : inclure/menu contient #SESSION dans un commentaire qui croit bien faire mais declenche le cache par session sur toutes les pages ou il est appelé (soit en general tout le site...).
(Multiplication des caches sur les sites qui utilisaient le plugin)
11 years ago
cedric@yterium.com 09589c47e8 Prise en charge des classes conventionneles de bootstrap pour la barre de navigation
=> passage en v1.3.0
etat test plutot que dev car il semble bien que c'est plus proche de la realite
11 years ago
rastapopoulos@spip.org 5eccb0a9ac Branchage menu menu 2 12 years ago