WIP: Amélioration affichage des résultats de recherche
publié le , mis à jour⚙️ Ceci est une proposition de changement de code.
Rendez-vous sur sa page Coderberg pour l'osculter.
- distance par rapport à la position actuelle
- affichage du 'département' au lieu de la 'région' dans les résultats
- séparation avec des ',' au lieu de ' ' pour une meilleur facilité de lecture
- ne pas afficher de texte redondant quand les 2 lignes affichent la même donnée (pour avoir moins à lire)
- affichage d'un 'title' avec toutes les informations pour pouvoir différencier certains résultats similaires
Avant:
Après:
laem
Quelques remarques :
- le doublement de la hauteur des résultats me semble problématique. On perd beaucoup en concision
- s'il y a des informations différentiantes à mettre dans le "title", je pense qu'il faut les mettre dans le corps. Ici le "Iowa" est très intéressant par exemple, et le E-U-A ensuite est redondant. Cet exemple montre qu'on merde sur les 3 résultats perçus comme identiques ici (enfin "on", peut-être photon ?)
- super pour les virgules
- la distance me semble intéressante, mais elle est inutilement précise ici je pense. Je me demande si on n'a pas déjà un bout de code pour simplifier la distance, peut-être dans le composant "Y aller" ?
pmiossec
@laem wrote in https://codeberg.org/cartes/web/pulls/1400#issuecomment-9577310:
- le doublement de la hauteur des résultats me semble problématique. On perd beaucoup en concision
Je partage cette crainte mais en ayant testé sur mobile, j'ai pas vraiment vu de problème. Ce qui est pour moi beaucoup plus frustrant en terme de UX et concision, ce sont les catégories qui sont affichées au dessus et qui souvent ne sont pas bien/suffisement filtrées (--je pense que le threshold est trop élevé--) et qui oblige à scroller pour aller aux résultats:
ex: Paris ou pire "inter" (pour trouver un "intermarché" qui donne le résultat 😱:
s'il y a des informations différentiantes à mettre dans le "title", je pense qu'il faut les mettre dans le corps. Ici le "Iowa" est très intéressant par exemple, et le E-U-A ensuite est redondant.
Très juste : j'ai vu qu'au moment de faire le screenshot de la PR (et flemme de tout devoir recommencer) et comme c'est déjà le comportement habituel et que je ne sais pas si c'est facile à corriger pour satisfaire tous les adresses de pays je pensais regarder plus tard...
la distance me semble intéressante, mais elle est inutilement précise ici je pense.
ok, je vais regarder mais je sais pas comment faire mieux. Pourquoi on utilise jamais
Mm, Mega-mètres 😅laem
Ce qui est pour moi beaucoup plus frustrant en terme de UX et concision, ce sont les catégories qui sont affichées au dessus et qui souvent ne sont pas bien/suffisement filtrées (--je pense que le threshold est trop élevé--)
Oui tout à fait, mais c'est un pb parallèle. C'est un problème connu depuis très longtemps, mais on ne l'a pas encore corrigé. Je pense qu'il y a un ticket qui en parle.
Pourquoi on utilise jamais Mm, Mega-mètres 😅
Haha oui ce serait pratique.
pmiossec
@laem wrote in https://codeberg.org/cartes/web/pulls/1400#issuecomment-9646355:
Oui tout à fait, mais c'est un pb parallèle. C'est un problème connu depuis très longtemps, mais on ne l'a pas encore corrigé. Je pense qu'il y a un ticket qui en parle.
👍Je vais pousser mon fix sur l'ajustement du threshold ce qui permettra de discuter...
etienneJr
@pmiossec wrote in https://codeberg.org/cartes/web/pulls/1400#issuecomment-9646211:
qui souvent ne sont pas bien/suffisement filtrées (--je pense que le threshold est trop élevé--)
il y a ptet de l'optimisation de threshold à faire, mais surtout, on devrait n'afficher les propositions de catégories que si il y en a 5 ou moins.