Produits Entr'ouvert » Intégrations graphiques Publik

Avec un commit pour chacun des points.

• supprimer la marge basse de la liste
• supprimer le trait souligné sous le dernier item

Voici une capture qui illustre mieux pourquoi je veux retirer la marge basse de la liste.

Dans la précédente capture, la marge basse de la liste (qui contre l'effet de la classe CSS no-bottom-margin ) était artificiellement gommée en ajoutant une balise <br/> dans la cellule.

Remarques prises en compte.

Je n'ai pas compris qu'elle capture correspond à après / avant.

Il s'agit de 2 captures "avant" sur la recette.
Je joins la capture "après" obtenue localement.

Mais j'ai en tout cas l'impression que ce patch correspond à une cellule en spécifique (et pas toutes)

bof, potentiellement toutes les cellules qui utiliseront "no-margin-bottom".
(je vais demander à Brice de faire un tour par là)

Pourquoi tu passes par des variables intermédiaires $secondary-color et $tertiary-color plutôt que d'utiliser directement les variables de couleur ($peru, $blue-dark, etc.) ?

Pour n'avoir à changer la couleur qu'à un seul endroit.
(si l'on décide de changer la couleur secondaire ou ternaire, alors il n'y a pas besoin de parcourir le thème pour changer la couleur ou le nom de la variable qui la représente).

(je vais demander à Brice de faire un tour par là)

Sur une cellule json qui listes les déclarations des prévenus, on utilise la classe "no-bottom-margin" pour accoler une cellule texte qui ajoute une information dans ce même contexte (un lien vers la démarche de déclaration de revenus), mais qui sera successible d'évoluer par la suite.
Mon CPF souhaitait avoir un bel affichage, tout en gardant la main sur la cellule texte pour faire évoluer le lien ou son texte, sans être tributaire du cycle des montées de versions (alors que la cellule prototype json sera convertie en cellule json pour le passage en production).

je serais donc pour rendre le patch spécifique à la cellule

tu me proposes de conditionner ces modifications sur les cellules proto-json et json uniquement ?

• ajouter une classe no-bottom-margin-for-minsant-exclusively
• assumer que les modifications du texte pourraient se faire en hotfix (sauf que non, ça je ne le peux pas)
• dire à mon CPF de ne pas utiliser no-bottom-margin

Discuté en réunion CPT : ajouter une variable (sous forme de paramètre à la cellule json) qui pointe sur le formulaire.

Même proposition sans 0005 (la cellule texte est fusionnée dans la cellule json et donc plus besoin de no-bottom-margin).

commit 709b9e8fdd2cc718fd0d506ca896ad0f6669c578
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Mar 10 14:34:11 2021 +0100

    minsant: enlarge folding cell link (#51864)

commit 9f33bb115583a8ba7848a2af4bcc1308216a7e50
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Mar 10 14:31:29 2021 +0100

    minsant: set peru color on wcs current step (#51864)

commit 7b50a6cbf97e8e42e8ed3f591a306dbf072be518
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Mar 10 14:28:55 2021 +0100

    minsant: reorder buttons and set peru color on submit (#51864)

commit 15f62c67d8097c109da9ee76ea4beb41bf334c2d
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Mar 10 14:25:45 2021 +0100

    minsant: set peru color on links (#51864)