Open emmanuelVentura opened 11 years ago
Je viens de voir que 3 commits sont joins par erreur, ils font déjà l'objet de pull-request. Désolé, j'apprends encore avec github :(
En effet si tu continues de commiter sur la même branche, la PR les prendra en compte, car c'est une demande de fusion de branches.
Heureusement les branches avec git ça coûte pas cher et ça permet de bien isoler les différents types de modifications.
C'est pour cette raison qu'il est recommandé de créer un branche par issue ou par feature, ici par exemple :
git checkout -b 87-forms-evolution
(reprendre le numéro d'issue de github est une convention possible, ça permet de s'y retrouver facilement).
Une fois les modification terminées, tu publies la branche et tu fais une pull request.
Tout est très bien expliqué ici : https://github.com/necolas/normalize.css/blob/master/CONTRIBUTING.md#pull-requests
Ajout de deux fonctionnalités au module formulaire :
Affichage sans tableau Un paramètre au niveau du module permet de définir la sortie du module formulaire. Par défaut rien ne change. Par contre, en modifiant ce paramètre, en le passant à "clean", le cms_form peut alors sortir tous les formulaires sans tableau mais avec un div autour de chaque couple label/champ. J'ai opté pour ne pas modifier l'enregistrement du formulaire et pour traiter le nettoyage de la chaine à la sortie. Cela permet de conserver la compatibilité.
Ajout d'un champ de type fieldset. Un nouveau type de champ est disponible dans le wizard : "Groupe de champs". Sur la base d'une chaine de caractère, ce champ permet d'insérer des titres dans les tableaux de mise en forme des formulaires. Cela évite de perdre la mise en page à chaque modification. Dans la vue sans tableau, ces champs se transforment en balise fieldset. A noter que ce champ n'est pas présent dans le mail envoyé.