Conversation
yaaax
left a comment
yaaax
left a comment
There was a problem hiding this comment.
Effet visuel au top ! Restitution aux aides techniques OK.
- 1 proposition de rework mineur de code
- 1 bug CSS (responsive)
- 1 bug JS (
<button>vs<div>et setTimeout)
| </div> | ||
| <div class="fr-btns-group fr-btns-group--icon-left"> | ||
| <CopyButton | ||
| label="Copier le lien de partage" |
There was a problem hiding this comment.
Et si on mettait ces 2 props label et success-label en valeurs par défaut dans le composant CopyButton à "Copier le lien de partage" et "Lien de partage copié" ? Comme ça pas besoin de répéter à chaque fois ces même textes. (En gardant la possibilité de spécifier d’autres labels au cas où…)
| <style scoped> | ||
| .copy-button { | ||
| &:not(.copy-button--within-btn-group) { | ||
| width: v-bind(initialButtonWidth) !important; |
There was a problem hiding this comment.
Super le width qui se met à jour !
Attention ça casse le côté responsive sur petits écrans, côté rapport ("within btn group").
Rajouter ici :
max-width: 100%;Et sur .heading-action :
flex-wrap: wrap;
max-width: 100%;| showSuccess.value = true; | ||
| }); | ||
|
|
||
| setTimeout(() => { |
There was a problem hiding this comment.
Attention le comportement est étrange si on clic alors que le message de succès est déjà affiché et que le délai (setTimeout) est en cours.
Après la sélection du bouton "Copier le lien de partage", changer le <button> par une <div> ? Ainsi il ne sera plus possible de cliquer dessus ! Attention à la restitution aux aides techniques.
Aussi, pour aller jusqu’au bout, on peut vérifier l’id retourné par le setTimeout : ne pas relancer de setTimeout tant qu’un délai est en cours.
3 participants
closes #1441