dnum-mi / vue-dsfr

Portage en Vue du DSFR
https://vue-ds.fr
Other
61 stars 28 forks source link

[DsfrSearchBar] prop label-visible non fonctionnel #597

Closed ajacque-trydea closed 1 year ago

ajacque-trydea commented 1 year ago

Le composant searchBar déclare une prop "label-visible" mais modifier celle-ci ne change rien au composant

possible fix :

<DsfrInput
      :id="id"
      type="search"
      :placeholder="placeholder"
      :model-value="modelValue"
      :label-visible="false" <!-- ------ :label-visible="label-visible" ------- --> 
      :label="label"
      :disabled="disabled"
      @update:model-value="$emit('update:modelValue', $event)"
      @keydown.enter="$emit('search')"
    />
laruiss commented 1 year ago

J’ai plutôt l’impression que ça fonctionne très bien. Regardez un exemple ici :

https://dsfr-input-label.netlify.app/

(un lien vers le code source est sur la page)

Si vous avez réellement écrit :label-visible="label-visible", vous risquez effectivement d’avoir des résultats assez aléatoires. Attendu que ce qu’il y a à l’intérieur des guillemets (label-visible) est une expression qui sera évaluée (puisque le nom de la prop est précédé du caractère :), le composant DsfrInput recevra un Number qui sera le résultat de la soustraction label - visible... Vraisemblablement toujours NaN, et donc ce sera évalué à false puisque la prop attend un booléen.

Si le problème persiste, merci de refaire un projet à partir de ce gabarit et de rouvrir l’issue avec un lien vers ce projet.

ajacque-trydea commented 1 year ago

Je n'ai pas été assez clair dans mon ticket, désolé. Ce n'est pas le dsfr Input qui pose problème mais le dsfr search bar. et l'extrait de code fourni présente un correctif à appliquer au fichier DsfrSearchBar.vue afin de corriger le problème.

laruiss commented 1 year ago

D’accord, mes excuses, je vais regarder ça !

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 4.7.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: