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

[FEATURE] Créer le nouveau menu (PIX-8994) #579

Merged
merged 6 commits into from
Oct 9, 2023

Conversation

clarissedmn
Copy link
Contributor

@clarissedmn clarissedmn commented Sep 27, 2023

🦄 Problème

Nous avons besoin d'un nouveau design pour le header et menu de navigation.

Lien vers les designs

🤖 Proposition

En fonction du feature toggle, afficher de nouveaux composants avec un nouveau design.

🌈 Remarques

💯 Pour tester

Aller sur la RA et vérifier l'affichage du nouveau menu.

  • Sur pix-site
  • sur pix-pro
  • selon les locales
  • responsive

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr579/environment

@clarissedmn clarissedmn force-pushed the pix-8984-create-new-navigation-menu branch 6 times, most recently from 9d7e0cd to 2eabff8 Compare September 28, 2023 17:32
@clarissedmn clarissedmn force-pushed the pix-8984-create-new-navigation-menu branch from 8c3f4bb to 8c2b4a6 Compare September 29, 2023 08:59
@Libouk
Copy link
Member

Libouk commented Oct 6, 2023

Les différences de texte/contenu, j'imagine que ce n'est pas dans le scope de la PR ? (Concernant Pix Emploi / Vous êtes, par exemple)

@Libouk
Copy link
Member

Libouk commented Oct 6, 2023

Je n'ai pas réussi à aller activer le feature toggle pour Pix Pro et donc pas pu tester sur Pix Pro

@clarissedmn clarissedmn force-pushed the pix-8984-create-new-navigation-menu branch from b0205f6 to b2dcd4e Compare October 6, 2023 14:35
components/slices/NavigationZoneV2.vue Outdated Show resolved Hide resolved
components/slices/NavigationZoneV2.vue Outdated Show resolved Hide resolved
Copy link
Contributor

@matthiasferraina matthiasferraina left a comment

Choose a reason for hiding this comment

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

lgtm 🚀

@clarissedmn clarissedmn force-pushed the pix-8984-create-new-navigation-menu branch 2 times, most recently from 8ef9f71 to 7f0a7e0 Compare October 9, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants