[FEATURE] Redesign du menu burger (PIX-2257). #249
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I'm deploying this PR to these urls:
Please check it out! |
alexandrecoin
added
👀 Func Review Needed
👀 Tech Review Needed
team-evaluation
alexandrecoin
force-pushed
the
pix-2257-responsive-navigation-with-connexion-button
branch
6 times, most recently
from
1f5b050 to
da50984
Compare
celineung
reviewed
Mar 3, 2021
There was a problem hiding this comment.
✅ sur IE
Il y a quelques comportements bizarres mais qui étaient déjà là avant.
Je note là mais c'est à traiter dans une autre PR:
- le burger menu ne se ferme pas lorsqu'un élément du menu a été cliqué et que l'utilisateur est redirigé vers la page demandé
- lorsque le burger est ouvert, l'opacité en background ne se met que sur la navigation
- certaines pages sont toutes cassées sous IE (ex: enseignement scolaire) 😢
celineung
approved these changes
Mar 3, 2021
alexandrecoin
force-pushed
the
pix-2257-responsive-navigation-with-connexion-button
branch
3 times, most recently
from
5c64928 to
fcf164e
Compare
MelanieMEB
approved these changes
Mar 4, 2021
sbedeau
suggested changes
Mar 4, 2021
alexandrecoin
force-pushed
the
pix-2257-responsive-navigation-with-connexion-button
branch
from
bbeea71 to
b5ac0f3
Compare
sbedeau
approved these changes
Mar 4, 2021
alexandrecoin
force-pushed
the
pix-2257-responsive-navigation-with-connexion-button
branch
from
b5ac0f3 to
3a88c77
Compare
alexandrecoin
added
Func Review OK
Tech Review OK
and removed
👀 Func Review Needed
👀 Tech Review Needed
labels
alexandrecoin
deleted the
pix-2257-responsive-navigation-with-connexion-button
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Le menu de navigation burger n'a pas été soumis au même traitement que le reste de Pix-Site et Pix-Pro.
🤖 Solution
Nouvelle intégration du menu pour mieux correspondre à ce qu'il est possible de retrouver sur les maquettes correspondantes.
🌈 Remarques
Plusieurs points sont à noter :
Nous avons choisi, et contrairement à ce qu'il est possible de retrouver sur les maquettes, de ne pas intégrer les logos de Pix et République Française car depuis la PR [FEATURE] Ne plus bouger les icone quand on ouvre le burger menu (PIX-2242). #247 , les icônes ne se déplacent plus lorsque l'on ouvre le menu, les laissant ainsi apparaitre une fois le menu ouvert, les rendant redondants. A cela s'ajoute une intégration non aisée liée à l'utilisation de la librairie
vue-burger-menu.Nous avons modifié l'organisation des données passées en
propsau composantBurgerMenuNavafin de les rendre plus facilement utilisables et de mieux accentuer la séparation entre ce estlienetaction(ex: Enseignement Scolaire VS S'inscrire).Cela nous incite toutefois à une répétition de la création de la liste
<ul>dans le composant de par la pluralité nouvellement créée des données passée à la liste.Nous optons donc pour la création d'un nouveau composant
BurgerMenuNavItemsListcontenant la liste<ul>en question et réutilisée deux fois dans le composantBurgerMenuNavafin de réduire la duplication de code.💯 Pour tester
Aller sur Pix-site (.fr et .org) et Pix-Pro, et vérifier que l'intégration correspond aux maquettes proposées.