search

popcorn-nantes / popcorn-machine

🍿La machine à Popcorn permet est un package NPM qui permet de créer un annuaire de dev freelances "Popcorn", tel que celui de Popcorn Nantes.
https://www.npmjs.com/package/popcorn-machine
MIT License
10 stars 8 forks source link

Suppression du double paint de la page au premier chargement #32

Closed yann-yinn closed 5 years ago

yann-yinn commented 5 years ago

Aujourd'hui, quand on se rend sur https://popcorn-nantes.github.io/, on peut observer pendant un court instant que les profils sont dans un certain ordre, avant d'etre écrasé par un autre ordre.

Ce fix évite ce "repaint" de la page au premier chargement. (régression suite à l'ajout de la fonctionnalité qui permet de conserver la recherche en revenant d'un profil

yann-yinn commented 5 years ago

Hello ! un test cypress qui teste que la recherche est bien conservée entre les pages tu veux dire ? ou pour vérifier qu'il n'y a pas 2 paints d'affilées ?

fabienjuif commented 5 years ago

Que la recherche est conservée et que l'ordre a pas changé (si j'ai bien suivi) ?

fabienjuif commented 5 years ago

Et le teste dans l'autre sens, que si pas de recherche, l'ordre change ?

yann-yinn commented 5 years ago

En effet il faut des tests cypress sur la recherche. Je vais quand même faire passer ce merge qui du genre "hotfix" car ça joue nettement sur les perfs mais je suis sur le front des test en parallèle