fix(look&feel): allow file upload to have desktop style without dropzone description

[a-pourrier]

Add withPadding parameter (default false) in order to not have the isMobile class on the component even without dropzoneDescription

[GuillaumeKESTEMAN]

Je crois qu'il n'y a juste pas besoin d'avoir cette vérification de description pour changer la place que prends le bouton.
De ce que je vois sur la maquette, tout les file upload qui ont un bouton quasiment en full width sont en vue mobile.

[JLou]

Je ne comprends pas l'objectif de la PR. Possible de détaillé plus avec des exemples avant/apres stp ?

[a-pourrier]

Je ne comprends pas l'objectif de la PR. Possible de détaillé plus avec des exemples avant/apres stp ?

Le but de la PR c'est de laisser le choix au dev du style du bouton importer, pour l'instant dès qu'on a pas de description de dropzone on a forcément le style mobile (voir ci dessous)

Alors que sur mon projet j'ai besoin d'un visuel plutôt comme ça :

[a-pourrier]

Je crois qu'il n'y a juste pas besoin d'avoir cette vérification de description pour changer la place que prends le bouton. De ce que je vois sur la maquette, tout les file upload qui ont un bouton quasiment en full width sont en vue mobile.

J'ai préféré rajouter un paramètre plutôt que juste casser le lien histoire d'éviter un breaking change pour les autres projets, mais c'est vrai que pour moi il faut pas lier les deux et juste mettre isMobile si on veut son style

[JLou]

Okay dans ce cas le parametre devrait plutot s'appeler withPadding pas isWithPadding. Ca serait plus correct niveau anglais

[GuillaumeKESTEMAN]

Je crois qu'il n'y a juste pas besoin d'avoir cette vérification de description pour changer la place que prends le bouton. De ce que je vois sur la maquette, tout les file upload qui ont un bouton quasiment en full width sont en vue mobile.

J'ai préféré rajouter un paramètre plutôt que juste casser le lien histoire d'éviter un breaking change pour les autres projets, mais c'est vrai que pour moi il faut pas lier les deux et juste mettre isMobile si on veut son style

Ok pour moi mais je pense qu'on profitera de l'arrivé d'apollo pour refacto ce composant car il en a besoin

[a-pourrier]

Okay dans ce cas le parametre devrait plutot s'appeler withPadding pas isWithPadding. Ca serait plus correct niveau anglais

C'est renommé