Skip to content

Rangement des menus#1338

Open
arnaudlevy wants to merge 15 commits intomainfrom
optimize-menu
Open

Rangement des menus#1338
arnaudlevy wants to merge 15 commits intomainfrom
optimize-menu

Conversation

@arnaudlevy
Copy link
Member

@arnaudlevy arnaudlevy commented Feb 12, 2026

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Rangement du menu, pour discussion et principes :

  • 35 lignes max par fichier
  • dossier helpers pour ranger les partials qui return quelquechose
  • limiter le nombre de paramètres envoyés
  • dupliquer plutôt que de mettre en option, surtout si c'est récursif

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

Migration

  • Vérifier spécifiquement les sites qui ont des menus social, une classe doit manquer.
  • Vérifier que l'absence de {{- if or (ne .stop 1) (and (eq .stop 1) (ne .kind "blank")) -}} ne dérange pas
  • Réparer le js pour les dropdowns de niveau 2 et 3
  • Faire attention à la suppression de classe disabledsur les items de type blank (les textes simples)

@arnaudlevy arnaudlevy changed the title refactor Rangement des menus Feb 12, 2026
@arnaudlevy arnaudlevy marked this pull request as draft February 12, 2026 23:03
@@ -1,7 +1,7 @@
{{ $menu := partial "GetMenu" "social" }}

{{ if $menu.items }}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ce test n'est pas ailleurs, pourquoi ?

@arnaudlevy arnaudlevy marked this pull request as ready for review February 13, 2026 10:55
arnaudlevy and others added 6 commits February 13, 2026 12:18
Co-authored-by: Alexis BENOIT <alex@noesya.coop>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant