Development #62751
ajouter un $widget-padding
0%
Description
L'intégration spécifique Rouen pose des règles particulières sur ses champs; ça peut être remplacé par l'utilisation de paramètres standards sauf pour le padding.
Fichiers
Révisions associées
haguenau: remove custom padding on button (#62751)
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-scss-add-widget-padding-to-control-widget-padding-62.patch 0001-scss-add-widget-padding-to-control-widget-padding-62.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est basique modulo la partie qui ajuste aussi le rendu des <select2>. (c'est testé avec clapotis et rouen).
Mis à jour par Thomas Jund il y a environ 2 ans
Cette idée a trainé sur la todoList et finalement, je l'ai supprimée.
C'est une mauvaise approche.
Pour moi il d'abord necessaire d'uniformiser la taille des boutons et des widgets.
Ou plutôt, donner aux widgets la taille (le padding) des boutons, pour permettre d'être coherent horizontalement quand input et bouton sont alignés sur la même ligne (cellule recherche, code de suivi, autre cas d'usages possibles).
Et même dans le cas de nom alignement horizontal, de garder un rythme vertical cohérent entre ligne d'input et boutons en fin de formulaire.
Et uniqument après, avoir éventuellement une variable qui fait varié le padding des widget + boutons en accord.
Avec la variable comme proposée ici, on ouvre la porte à encore plus déséquilibrer input et boutons et aux créateurs de thème de ne pas du tout prendre en compte ce facteur.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier padding-sur-boutons.png padding-sur-boutons.png ajouté
J'en ai besoin pour Rouen; l'alternative est d'y laisser n'importe quoi spécifique et partiel et buggué comme aujourd'hui, et je n'ai pas envie.
J'ai tenté d'appliquer $widget-padding aux boutons et c'est tout à fait ok pour Rouen (qui a un gros padding) mais ça n'est pas approprié ailleurs, cf capture clapotis, sûr ça donne aux boutons la même hauteur qu'aux champs mais c'est trop petit (je trouve).
Alternativement, si on ne veut pas ici, tant pis, et je proposerai séparément les adaptations select2, qu'au moins il n'y ait qu'une valeur à appliquer côté Rouen.
Mis à jour par Thomas Jund il y a environ 2 ans
mais c'est trop petit (je trouve).
Oui je trouve aussi, c'est pourquoi je pense qu'il faut mettre les paddings des boutons aux widgets par défaut dans le core (et pas l'inverse). Nos widget sont petits, ils gagneraient à s'uniformiser vers les boutons (et pas les boutons vers les widgets).
Mis à jour par Thomas Jund il y a environ 2 ans
--- a/static/includes/_forms.scss
+++ b/static/includes/_forms.scss
@@ -19,6 +19,7 @@ $widget-focus-outline: none !default;
$widget-focus-outline-offset: 0 !default;
$widget-box-shadow: null !default;
$widget-focus-box-shadow: none !default;
+$widget-padding: 0.625rem 1rem !default;
$button-focus-outline: $widget-focus-outline !default;
$button-focus-outline-offset: $widget-focus-outline-offset !default;
@@ -178,7 +179,7 @@ div.buttons div {
background: $button-background;
text-align: center;
color: $button-color;
- padding: 10px 15px;
+ padding: $widget-padding;
cursor: pointer;
border: $button-border;
border-radius: $button-border-radius;
@@ -248,7 +249,7 @@ input, input[type="search"], textarea, select {
border: $widget-border;
border-radius: $widget-border-radius;
margin: 0.2em 0px;
- padding: 0.4em 0.7em 0.4em 0.7em;
+ padding: $widget-padding;
max-width: 100%;
transition: background 0.5s ease, color 0.5s ease, border 0.5s ease;
box-shadow: $widget-box-shadow;
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-scss-add-widget-padding-to-control-widget-padding-62.patch 0001-scss-add-widget-padding-to-control-widget-padding-62.patch ajouté
J'ai pris ça comme un message pour dire "j'ai testé la valeur $widget-padding: 0.625rem 1rem sur différentes intégrations et je trouve que ça fonctionne bien".
J'ai posé 0.625rem 1rem dans mon patch, étendu aux boutons, et j'ai validé le rendu select2 sur clapotis.
Mis à jour par Thomas Jund il y a environ 2 ans
- Fichier 0001-haguenau-remove-custom-padding-on-button.patch 0001-haguenau-remove-custom-padding-on-button.patch ajouté
- Statut changé de Solution proposée à Solution validée
Bon, j'ai pas relevé de gros problèmes.
Je propose d'ajouter ce petit patch pour haguenau.
Et allons-y comme ça.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
Go,
commit 3502028062f12fa33ab1601bc2654615d4118ec9
Author: Thomas JUND <tjund@entrouvert.com>
Date: Wed Mar 16 11:23:50 2022 +0100
haguenau: remove custom padding on button (#62751)
commit 0c5e64094762d6d78b82bb08460dc63e73e95bc1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat Mar 12 23:31:43 2022 +0100
scss: add $widget-padding to control widget padding (#62751)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: add $widget-padding to control widget padding (#62751)