codegouvfr / codegouv-web-poc

0 stars 0 forks source link

ExploreCatalog page, with base filtering logic #24

Closed mineraux closed 1 year ago

mineraux commented 1 year ago

Close #21

Features

Screen record

https://www.loom.com/share/88bc40a776f74b97be982b9969562fde

mineraux commented 1 year ago

Après test sur la branche, le comportement général est nickel, merci ! 1 bug majeur :

  • [ ] Le changement de tri fait planter l'app 1 bug mineur :
  • [ ] La flèche sur le bouton des filtres est inversée Améliorations :
  • [ ] Sur la page d'accueil Explore, est-il possible de gérer le retour arrirère du navigateur après un clic sur le bouton de recherche qui redirige vers la page Catalogue ?
  • [ ] Est-il possible de'ajouter, en plus du clic, la recherche à l'appui sur la touche Entrée ?

Aussi, je crois que l'on en avait parlé, mais je ne me souviens plus de la réponse ; est-ce que les composants seront à terme dans le storybook ?

Merci d'avance ! :)

Pour le bug je m'en suis rendu compte après coup, c'est corrigé dans la prochaine PR. Est-ce que ça te vas si on merge cette PR et je reporte toutes tes remarques dans l'issue la branche d'amélioration de la page #25 ? En ce qui concerne les Story, c'est justement fait dans cette issue #25 :)

hjonin commented 1 year ago

Pour le bug je m'en suis rendu compte après coup, c'est corrigé dans la prochaine PR.

Ok, super !

Est-ce que ça te vas si on merge cette PR et je reporte toutes tes remarques dans l'issue la branche d'amélioration de la page #25 ?

Oui ça me va !

En ce qui concerne les Story, c'est justement fait dans cette issue #25 :)

Super également :)