Development #68106
Ajouter des classes BEM sur les menus
Début:
10 août 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Je trouve utile de pouvoir faire simplement la différence menu / submenu. A voir s'il y a une meilleure manière d'organiser les classes et détecter si on est dans un menu ou un sous-menu.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Corentin Séchet il y a plus d'un an
- Fichier 0001-general-add-BEM-classes-on-menus-68106.patch 0001-general-add-BEM-classes-on-menus-68106.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Corentin Séchet il y a plus d'un an
- Lié à Development #64513: Ajouter des classes BEM sur la navigation ajouté
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
Je pense qu'il est posible d'être plus simple côté class.
Ta proposition :
.menu
.menu-item
.menu-item--link
.submenu
.submenu-item
.submenu-item--link
Je pense qu'il est inutile de jongler entre 2 block BEM: menu et menu-item, menu peut suffire. À mon avis, inutile d'indiquer via BEM que link est un enfant de item.
.menu
.menu--item
.menu--link
.submenu
.submenu--item
.submenu--link
Mis à jour par Corentin Séchet il y a plus d'un an
- Fichier 0001-general-add-BEM-classes-on-menus-68106.patch 0001-general-add-BEM-classes-on-menus-68106.patch ajouté
- Statut changé de Solution validée à Solution proposée
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Corentin Séchet il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 39d679a4717bb6fe7eeab69e24fe19dc876f793f
Author: Corentin Séchet <csechet@entrouvert.com>
Date: Wed Aug 10 17:38:02 2022 +0200
general: add BEM classes on menus (#68106)Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
general: add BEM classes on menus (#68106)