Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout d'un "top menu" au header #702

Merged
merged 13 commits into from
Oct 25, 2024
Merged

Ajout d'un "top menu" au header #702

merged 13 commits into from
Oct 25, 2024

Conversation

Olivia206
Copy link
Contributor

@Olivia206 Olivia206 commented Oct 25, 2024

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Passage des overrides spécifiques en option dans le thème.
Ajout d'un title="" aux lien du header.

Le top menu :

  • n'a qu'un premier niveau
  • peut accueillir la recherche (cids/mids)
  • se base sur les variables du header (ex: la bordure, pour sufficiency)
  • n'affiche pas l'icon des liens externes (ajout d'un paramètre dans le partial "menu"), mais le title oui (a11y)

Pour le mobile, le menu supérieur doit se trouver dans le dropdown (cf. screenshots), pour réussir à l'afficher au-dessus de l'ensemble logo + menu, je dois donc faire un trick qui mériterait d'être amélioré : si j'ai un top-menu, alors je n'affiche pas le logo normal (en desktop), j'ajoute un logo dans la div .menu histoire de faire jouer les order et passer le top-menu en haut. En mobile c'est l'inverse, je masque le logo qui est dans .menu afin que tout reprenne son comportement normal.

Je me suis permis ça car c'est un besoin ponctuel et qu'on en a besoin là de suite, mais il y a sans doute des méthodes qui n'impliqueraient pas d'avoir des images chargées !

❓ À titre perso je trouve "top menu" pas super élégant, des suggestions ? Upper est plus joli, non ?

Niveau d'incidence

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

Référence (ticket et/ou figma)

#652

Tests

Homepage des sites

  • example avec la branche example-with-top-menu
  • CIDS/MIDS
  • Sufficiency
  • Degrowth

Screenshots

Capture d’écran 2024-10-25 à 16 41 51
Capture d’écran 2024-10-25 à 16 41 40

Capture d’écran 2024-10-25 à 16 19 57
Capture d’écran 2024-10-25 à 16 20 06

@Olivia206 Olivia206 added enhancement New feature or request Incidence faible 😌 Risque faible de régression sur les sites labels Oct 25, 2024
@Olivia206 Olivia206 marked this pull request as ready for review October 25, 2024 14:52
@Olivia206 Olivia206 merged commit 570824e into main Oct 25, 2024
2 of 3 checks passed
@Olivia206 Olivia206 deleted the top-menu branch October 25, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Incidence faible 😌 Risque faible de régression sur les sites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant