leplessistrevise: fix background glitch on summary page (#71606) #35

Merged
pmarillonnet merged 1 commits from wip/71606-leplessistrevise-form-history-glitch into main 2022-11-28 09:09:17 +01:00
Owner

J’attends juste la validation côté client que le résultat est convenable.
Visuellement ça donne ça.

J’attends juste la validation côté client que le résultat est convenable. Visuellement ça donne ça.
fpeters reviewed 2022-11-23 14:33:27 +01:00
fpeters left a comment
Owner

dans l'objet du commit : "fitch" (je comprends que c'est "fix a glitch" mais tes doigts vont trop vite).

dans l'objet du commit : "fitch" (je comprends que c'est "fix a glitch" mais tes doigts vont trop vite).
pmarillonnet force-pushed wip/71606-leplessistrevise-form-history-glitch from 6bb7c53d03 to 982b74a12e 2022-11-23 14:37:02 +01:00 Compare
Author
Owner

dans l'objet du commit : "fitch" (je comprends que c'est "fix a glitch" mais tes doigts vont trop vite).

Bien vu, merci, c’est corrigé :)

> dans l'objet du commit : "fitch" (je comprends que c'est "fix a glitch" mais tes doigts vont trop vite). Bien vu, merci, c’est corrigé :)
tjund requested changes 2022-11-23 14:50:02 +01:00
tjund left a comment
Owner

Pour ça, c'est mieux d'utiliser $form-background

Pour ça, c'est mieux d'utiliser `$form-background`
pmarillonnet changed title from [WIP] leplessistrevise: fitch background glitch on summary page (#71606) to [WIP] leplessistrevise: fix background glitch on summary page (#71606) 2022-11-23 14:51:32 +01:00
pmarillonnet changed title from [WIP] leplessistrevise: fix background glitch on summary page (#71606) to leplessistrevise: fix background glitch on summary page (#71606) 2022-11-23 14:53:36 +01:00
pmarillonnet force-pushed wip/71606-leplessistrevise-form-history-glitch from 982b74a12e to 02bc6bf2a3 2022-11-23 14:56:17 +01:00 Compare
Author
Owner

Pour ça, c'est mieux d'utiliser $form-background

C’est fait, merci. J’ai tout de même laissé la partie

+#evolution-log ul#evolutions > li::after {
+       background: none;
+}

pour le retrait du tiret horizontal gris qui se superposait au texte de façon peu élégante.

> Pour ça, c'est mieux d'utiliser `$form-background` C’est fait, merci. J’ai tout de même laissé la partie <pre> +#evolution-log ul#evolutions > li::after { + background: none; +} </pre> pour le retrait du tiret horizontal gris qui se superposait au texte de façon peu élégante.
pmarillonnet requested review from tjund 2022-11-23 14:56:57 +01:00
tjund requested changes 2022-11-23 15:46:33 +01:00
tjund left a comment
Owner

Non, il faut donner à la variable la couleur que tu souhaites en bg de #rub-services :

// custom l.190
div#rub_service {
-	background: darken(white, 3%);

// dans vars.scss
+ $form-background: darken(white, 3%);
Non, il faut donner à la variable la couleur que tu souhaites en bg de #rub-services : ``` // custom l.190 div#rub_service { - background: darken(white, 3%); // dans vars.scss + $form-background: darken(white, 3%); ```
pmarillonnet force-pushed wip/71606-leplessistrevise-form-history-glitch from 02bc6bf2a3 to 7f84ab7e92 2022-11-23 15:54:59 +01:00 Compare
Author
Owner

Non, il faut donner à la variable la couleur que tu souhaites en bg de #rub-services :

// custom l.190
div#rub_service {
-	background: darken(white, 3%);

// dans vars.scss
+ $form-background: darken(white, 3%);

Ah désolé j’avais pas vu que je tapais cette ligne contradictoire dans la css custom. C’est corrigé et le patch est bien plus succinct.

> Non, il faut donner à la variable la couleur que tu souhaites en bg de #rub-services : > > ``` > // custom l.190 > div#rub_service { > - background: darken(white, 3%); > > // dans vars.scss > + $form-background: darken(white, 3%); > ``` Ah désolé j’avais pas vu que je tapais cette ligne contradictoire dans la css custom. C’est corrigé et le patch est bien plus succinct.
pmarillonnet requested review from tjund 2022-11-23 15:57:12 +01:00
tjund approved these changes 2022-11-23 15:59:51 +01:00
pmarillonnet merged commit 9bd3e3a016 into main 2022-11-28 09:09:17 +01:00
pmarillonnet deleted branch wip/71606-leplessistrevise-form-history-glitch 2022-11-28 09:09:17 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#35
No description provided.