proposition of some fixes #1

tonitch · 2024-04-05T09:47:06+02:00

tonitch commented

2024-04-05 09:47:06 +02:00

voilà en gros les changements que je ferais. c'est pas obligatoire mais c'est parce que tu n'utilise pas les fonctionnalitées de vue.

J'ai aussi un problème avec la fonction inputkeyup() parce que ça se voit clairement que tu l'as trouvée sur w3 et tu n'utilise pas du tout vue alors qu'en vue ça prendrais deux ligne mais c'est compréhensible du coup à voir si tu veux changer ça ou laisser comme ça

voilà en gros les changements que je ferais. c'est pas obligatoire mais c'est parce que tu n'utilise pas les fonctionnalitées de vue. J'ai aussi un problème avec la fonction `inputkeyup()` parce que ça se voit clairement que tu l'as trouvée sur w3 et tu n'utilise pas du tout vue alors qu'en vue ça prendrais deux ligne mais c'est compréhensible du coup à voir si tu veux changer ça ou laisser comme ça

tonitch added 1 commit 2024-04-05 09:47:06 +02:00

proposition of some fixes b8b193f344

Maxime commented

2024-04-05 10:25:37 +02:00

ça me parait vrm bien mieux et c'est ce vers quoi je voulais tendre au final merci

au nivo du inputkeyup() mettre un v-model et un @change comme pour les stats te semble plus judicieux ?

ça me parait vrm bien mieux et c'est ce vers quoi je voulais tendre au final merci au nivo du `inputkeyup()` mettre un v-model et un @change comme pour les stats te semble plus judicieux ?

tonitch commented

2024-04-05 10:32:18 +02:00

ça me parait vrm bien mieux et c'est ce vers quoi je voulais tendre au final merci

Désolé j'aurais pu t'aider plus tot, j'avais pas compris que c'étais ça qui dérangait, mais au moins la différence peut t'aider à comprendre peu être ?

au nivo du inputkeyup() mettre un v-model et un @change comme pour les stats te semble plus judicieux ?

y'a pas que ça. perso je ferais en sorte que tes inputs soient dans une array (les noms genre noms = ['Adèle', 'Anies', ...] et puis utiliser un <li v-for="n in noms"><a>{{ n }}</a></li>
Et finalement ta fonction inputkey serait au final une fonction filtre genre searchInList(maliste, value) => retourne une liste des éléments de maliste qui contiennent value et puis tu change le "noms" dans le v-for juste au dessus pour utiliser cette fonction
La magie de vue c'est que ça va se mettre à jour dynamiquement ^^

ps: c'est pas très important mais si tu veux que ta barre de recherche soit un peut cool du genre elle laisse passer les petites erreures de frappes tu peux utiliser la distance des mots https://en.wikipedia.org/wiki/Levenshtein_distance

> ça me parait vrm bien mieux et c'est ce vers quoi je voulais tendre au final merci Désolé j'aurais pu t'aider plus tot, j'avais pas compris que c'étais ça qui dérangait, mais au moins la différence peut t'aider à comprendre peu être ? > au nivo du `inputkeyup()` mettre un v-model et un @change comme pour les stats te semble plus judicieux ? y'a pas que ça. perso je ferais en sorte que tes inputs soient dans une array (les noms genre noms = ['Adèle', 'Anies', ...] et puis utiliser un `<li v-for="n in noms"><a>{{ n }}</a></li>` Et finalement ta fonction inputkey serait au final une fonction filtre genre `searchInList(maliste, value) => retourne une liste des éléments de maliste qui contiennent value` et puis tu change le "noms" dans le v-for juste au dessus pour utiliser cette fonction La magie de vue c'est que ça va se mettre à jour dynamiquement ^^ ps: c'est pas très important mais si tu veux que ta barre de recherche soit un peut cool du genre elle laisse passer les petites erreures de frappes tu peux utiliser la distance des mots https://en.wikipedia.org/wiki/Levenshtein_distance

Maxime merged commit ec2b975467 into master

2024-04-07 14:59:13 +02:00

Maxime referenced this issue from a commit

2024-04-07 14:59:15 +02:00

Merge pull request 'proposition of some fixes' (#1) from PGL/Clyde:tonitch/max/front/fix/vue into master

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Maxime/Clyde#1