Amélioration de la recherche par texte sur le tableau de suivi

[Ynote]

cf. Carte Trello : https://trello.com/c/DW8sFUe9/230-am%C3%A9liorer-le-filtre-texte-accents-et-majuscules

• Utiliser un index de tous les dossiers plutôt que créer un index par dossier à chaque recherche
• Gérer la recherche avec le numéro de dossier DS (cf. Not able to search for just numbers in lunr.de MihaiValentin/lunr-languages#66)

[DavidBruant]

Sans avoir lu le code, j'ai testé en local et je note les choses suivantes :

• on s'abstrait bien des accents/majuscules, c'est cool !
• la recherche ne cherche pas dans l'id pitchou
• la recherche cherche bien dans les communes, mais pas/mal dans les départements (attention à la corse et aux DOM aussi)
• Au-dessus du tableau, il reste encore les filtres communes/départements et on peut les enlever si la recherche textuelle gère ces cas... en continuant dans cette direction, je me demande si on ne peut pas juste mettre un <input> direct (plutôt qu'un bouton sur lequel il faut cliquer avant de pouvoir voir l'input)
• Je pense que ça serait cool si la recherche textuelle cherchait aussi dans l'activité principale (vu que c'est une nouvelle colonne du tableau)

[DavidBruant]

Trop cool dans l'ensemble

Et voir les commentaires pour quelques changements (dont certains, ça pourrait être ok de les faire dans des PR séparées si t'as pas la foi de les faire dans celle-ci)

[DavidBruant]

Je n'ai pas vu de réponse ou de réaction à ce que j'avais écris précédemment

Au-dessus du tableau, il reste encore les filtres communes/départements et on peut les enlever si la recherche textuelle gère ces cas... en continuant dans cette direction, je me demande si on ne peut pas juste mettre un direct (plutôt qu'un bouton sur lequel il faut cliquer avant de pouvoir voir l'input)

Est-ce que c'est un oubli ou un truc à papoter ?

[Ynote]

Est-ce que c'est un oubli ou un truc à papoter ?

Totalement un oubli, sorry ! Je suis d'accord avec ta proposition d'un input toujours affiché.

[Ynote]

@DavidBruant J'ai fait tes retours. Je veux bien à nouveau une review dessus, j'ai l'impression que c'est mieux là.

[DavidBruant]

c'est un détail, pas forcément besoin de le corriger dans cette PR, mais dans svelte 5, il n'y aura plus d'event
à la place, on passe une props avec une fonction (et cette fonction est appelée par le composant en remplacement des dispatch)

[Ynote]

Ah ! En plus, je crois que tu me l'as déjà mentionné. Ok je vais faire ça dans une PR suivante dans la foulée et voir si on a des dispatch ailleurs.

[Ynote]

cf. #114

[DavidBruant]

ouh lala ! J'adore avec l'input direct 🤩

et c'est trop cool la petite ligne de documentation juste en-dessous qui indique ce qui va marcher

à part pour l'histoire de la regexp, tout est bon pour moi
Si c'est bon pour toi après la correction de la regexp, c'est bon pour moi pour merger !

Merci beaucoup !!

[Ynote]

ouh lala ! J'adore avec l'input direct 🤩

Ouais moi aussi j'aime bien. C'est cool, j'ai hâte que ce soit testé par les instructrices :)