Development #68612
permettre une couleur de fond des démarches différente de celle du corp de la page
0%
Description
Des fois il est souhaité que la couleur de fond des formulaires soit différente de celle du body.
On doit avoir une variable qui permet de faire cela.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus d'un an
- Lié à Bug #68609: lille-metropole-2022: le fond du formulaire doit être blanc ajouté
Mis à jour par Serghei Mihai il y a plus d'un an
- Fichier 0001-scss-add-variable-to-define-form-s-background-color-.patch 0001-scss-add-variable-to-define-form-s-background-color-.patch ajouté
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus d'un an
Couleur de fond du formulaire.
des démarches, pour ne pas laisser croire que ça s'appliquerait aussi aux formulaires type création de compte ou autre. (ou alors le patch demande à être revu).
Mis à jour par Serghei Mihai il y a plus d'un an
- Sujet changé de permettre une couleur de fond des formulaires différente de celle du corp de la page à permettre une couleur de fond des démarches différente de celle du corp de la page
J'entendais les démarches, en effet.
Mis à jour par Frédéric Péters il y a plus d'un an
(je parlais de la documentation intégrée dans le patch proposé)
Mis à jour par Serghei Mihai il y a plus d'un an
- Fichier 0001-scss-add-variable-to-define-form-s-background-color-.patch 0001-scss-add-variable-to-define-form-s-background-color-.patch ajouté
Et la doc corrigée.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
- Statut changé de Solution proposée à En cours
Attention.
Patch plus complèxe qu'il n'y parait pour avoir commencer à me pencher dessus et liasser de côté.
j'explique ensuite.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
Ce qui attire l'attention c'est pourquoi avoir poser
div#rub_service { background: $body-background; }
alors que ne rien mettre aurait donner le même résultat (avec un background transparent qui laisse apparaitre le bg du body) ?
La réponse se trouve dans la façon dont est construit le composant `ul#evolutions`, il utilise $body-background pour créer des masques.
ul#evolutions div.evolution-metadata, ul#evolutions li div.msg { $body-background }
La solution la plus simple est d'appliquer $form-background ici aussi.
(Mais cela ne règle pas le fait qu'on a aussi parfois envie de donner un background à main, à #content, à form.quixote. La bonne voie serait de refactoriser le module d'évolution pour pouvoir appliquer des backgrounds comme bon nous semble. Je dois reprendre mon patch en attente qui propose une refacto, c'est l'inspection de l'existant que je n'ai pas encore terminé).
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
(Quand je dis « La solution la plus simple est d'appliquer $form-background ici aussi » c'est que ça me va bien pour avancer comme ça)
Mis à jour par Serghei Mihai il y a plus d'un an
- Fichier 0001-scss-add-variable-to-define-form-s-background-color-.patch 0001-scss-add-variable-to-define-form-s-background-color-.patch ajouté
- Statut changé de En cours à Solution proposée
Ok, j'applique $form-background au bloc des evolutions aussi.
Mis à jour par Serghei Mihai il y a plus d'un an
Mis à jour par Corentin Séchet il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
commit b0d93e382fcbc68105fa136ccb73c49afb73786a Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Sep 1 11:02:06 2022 +0200 scss: add variable to define form's background color (#68612)
Mis à jour par Serghei Mihai il y a plus d'un an
Et Thomas me rappelle qu'aujourd'hui c'est la màj des prods, je n'aurais pas dû pousser :-\
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
scss: add variable to define form's background color (#68612)