feat(DsfrModal): ✨ changer la taille de la modale #641
Conversation
👷 Deploy request for vue-dsfr pending review.Visit the deploys page to approve it
|
| class="fr-col-12" | ||
| :class="{ | ||
| 'fr-col-md-8': size === 'lg', | ||
| 'fr-col-md-6': size === 'md', | ||
| 'fr-col-md-4': size === 'sm', | ||
| }" |
There was a problem hiding this comment.
Pourquoi fr-col-12 serait toujours présent ?
Pourquoi pas ceci :
<div
:class="{
'fr-col-md-12': size === 'xl',
'fr-col-md-8': size === 'lg',
'fr-col-md-6': size === 'md',
'fr-col-md-4': size === 'sm',
}"
```
There was a problem hiding this comment.
jusqu'à présent je crois que c'était pour le mode mobile pour que la modale ne soit pas trop petite et prenne tout l'écran dans ce cas là
There was a problem hiding this comment.
D'après le dsfr: modal simple fr-col-12 fr-col-md-8 fr-col-lg-6
Modal SM fr-col-12 fr-col-md-4
Modal LG fr-col-12 fr-col-md-8
https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/modale
| }, | ||
|
|
||
| computed: { | ||
| modifyActions () { |
There was a problem hiding this comment.
Ce serait pas plutôt modifiedActions ?
There was a problem hiding this comment.
yes bien vu, au début on était parti sur une méthode mais après on a fait un computed et on a oublié de renommé
Je change ça 😁
abraccoippon
force-pushed
the
develop
branch
from
3fb192e to
62b7739
Compare
|
🎉 This PR is included in version 5.2.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Resolve #638