arnaudpant / vinted

Vends Le: clone de vinted
https://bootcamp-vinted.netlify.app/
2 stars 2 forks source link

Feature/header banner #8

Closed nabilCodeur closed 8 months ago

nabilCodeur commented 8 months ago

C'est le code des deux parties du header, et de la bannière (avec la photo de la dame). Apparement le header est sur toutes les pages, donc je fais appel directement à ce composant dans App. Le reste de la page change en fonction de la route.

La bannière est dans Home. J'ai profité pour ajouter le footer dans Home.

Aussi le header est fixe lors du scroll , j'ai utilisé les propriétés positions pour le rendre possible.

Dernière chose, lors du clic sur la navBar de catégoris (Femmes,enfants), une pop-up de sous-catégories s'ouvre. La position de la popup horizontale change en fonction de la catéogie sélectionnée. J'ai pu avoir ette fonctionnalité avec un listener qui détermine la position hozitontale de la position sélectionnée. Le problème conséquent est qu'il n'est pas possible de fermer la pop-up en cliquant à gauche de celle-ci. Il est peut-être possible de faire autrement sans listner, avec un flex, mais j'ai préféré avancer.

ghorreso commented 8 months ago

Je continuerai la review demain (= mercredi)

ghorreso commented 8 months ago

Pas encore fini la review, je continue ce soir si je peux

ghorreso commented 8 months ago

PR ouverte via le fork de Nabil clos car la même branche a été créée sur le repo principal et une autre PR a été ouverte