feat: rework the caterogies search with fuse.js
publié le , mis à jour⚙️ Ceci est une proposition de changement de code.
Rendez-vous sur sa page Coderberg pour l'osculter.
n4n5
https://codeberg.org/cartes/web/pulls/2625#issuecomment-20761967
Ça marche pour cet exemple, mais c'est parce que tu rends l'algorithme plus tolérant, tout simplement. Il se pourrait que ça fasse apparaitre des catégories qui n'ont que peu à voir (faux positifs), non ?
laem
Ahlala, notre recherche côté client est vraiment pétée. Depuis quand on affiche un résultat religieux d'un côté, et un résultat de riches d'un autre, avant la catégorie recherchée ? J'ai vérifié, rien à voir avec les modifs récentes de Fuse. Ça fait longtemps donc.
Y a du boulot 😓
etienneJr
@laem wrote in https://codeberg.org/cartes/web/pulls/2717#issuecomment-21315650:
Depuis quand on affiche un résultat religieux d'un côté, et un résultat de riches d'un autre, avant la catégorie recherchée ?
il y aurait un fix tout simple pour ce cas précis, ça serait de les renommer en "halal" et "étoilé" sans le mot restaurant, qui est de toute façon inutile vu qu'on est dans le groupe restaurant et que l'icone est claire. Mais attention, ça va modifier la
keyde ces 2 catégories, donc il faudra modifier en même temps tous les fichiers de traduction. (c'était pour éviter ça que, il y a qq mois, j'avais proposé de définir des keys qui soient indépendantes du name...)laem
(dslé j'ai pas de temps pour ce sujet, j'aimerais bien bosser dessus, mais pas maintenant)
n4n5
Bon je suis entrain de ré-écrire cette MR
n4n5
Cleanup (type, refactor) https://codeberg.org/cartes/web/pulls/2999 avant une tentative
n4n5
un autre cleanup
https://codeberg.org/cartes/web/pulls/3001
je pense qu'il faudrait un CategoryMenu et un CategoryMenuSearch
actuellement c'est les meme composant, et si je dis pas de bétises, les catégories sont triés par ordre de...couleur ?!
Je trouve pas ça logique l'ordre de couleur sur une recherche de catégorie
n4n5
Exemple concret
Je pense que ici,
Pour moi, ça devrait être comme ça: les catégories qui matchent le plus en haut (ceci est un croquis, pas codé..encore)
@etienneJr @laem vous être d'accord ?
esmenard
Avec auto-école tout en haut, c'est la seule catégorie dont le nom commence par auto
n4n5
@esmenard wrote in https://codeberg.org/cartes/web/pulls/2717#issuecomment-22685743:
Avec auto-école tout en haut, c'est la seule catégorie dont le nom commence par auto
Pour moi non, le nombre de match exact d'abord, ici les services ont deux catégories qui match
n4n5
C'est en ligne, il faudra pimper un peu le truc peut etre
Il faudra ensuite:
- revoir fileredCategoriesMaxCount
- mettre vraiement le fusing (virer les regex), qui etait le premier but de cette MR
Et ça fix le pb de @laem