Development #26806
gouttière grille vs champs de formulaire
0%
Description
Les champs ont déjà un padding-right: 1rem qui assure un espace entre eux, la gouttière s'y ajoute inutilement.
Un truc comme ça à faire et évaluer sur les différentes combinaisons :
--- a/static/includes/_grid.scss +++ b/static/includes/_grid.scss @@ -11,7 +11,6 @@ form.quixote div[class*=grid-] { div[class*=grid-] { float: left; - box-sizing: border-box; padding-right: 1em; @media screen and (max-width: $very-small-limit) { width: 100%;
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-scss-don-t-cumulate-form-gutters-26806.patch 0001-scss-don-t-cumulate-form-gutters-26806.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Évidemment le faire là a des conséquences partout mais il y a moyen de switcher les champs de formulaire mis en grille vers box-sizing: content-box pour obtenir quelque chose qui me semble marcher partout.
Ça fournit à mon sens aussi une correction pour #25257 (dont le titre parle de cellules mais qui en fait concerne le rendu des formulaires).
L'alternative serait de ne pas toucher aux width, d'utiliser du padding, et de déplacer la bordure autour des cellules à leur div intérieur (passer de div.cell à div.cell > div). Ça me semble plus simple mais ça se heurte par contre à quantité de règles des intégrations existantes, qui agissent en considérant que la bordure est sur le div.cell (pour en changer le style, par exemple).
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-scss-don-t-cumulate-form-gutters-26806.patch 0001-scss-don-t-cumulate-form-gutters-26806.patch ajouté
Nouvelle version qui respecte le paramétrage de $grid-gutter pour les champs de formulaire sauf quand il est mis à 0px, où un espace est préservé. (sur l'idée qu'on veut pouvoir coller des cellules dans combo pour des questions de présentation mais qu'on n'a jamais envie de coller les champs bordure contre bordure).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b8ab118eb4153b0e8e02ee9149724d3f5f389148 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Oct 7 17:26:51 2018 +0200 scss: don't cumulate form gutters (#26806)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: don't cumulate form gutters (#26806)