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

Merged
pmarillonnet merged 1 commits from wip/71606-leplessistrevise-form-history-glitch into main 2 months ago
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.
pmarillonnet added 1 commit 3 months ago
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
6bb7c53d03
[WIP] leplessistrevise: fitch background glitch on summary page (#71606)
fpeters reviewed 3 months ago
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 3 months ago
Poster
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 3 months ago
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) 3 months ago
pmarillonnet changed title from [WIP] leplessistrevise: fix background glitch on summary page (#71606) to leplessistrevise: fix background glitch on summary page (#71606) 3 months ago
pmarillonnet force-pushed wip/71606-leplessistrevise-form-history-glitch from 982b74a12e to 02bc6bf2a3 3 months ago
Poster
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 3 months ago
tjund requested changes 3 months ago
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 3 months ago
Poster
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 3 months ago
tjund approved these changes 3 months ago
pmarillonnet merged commit 9bd3e3a016 into main 2 months ago
pmarillonnet deleted branch wip/71606-leplessistrevise-form-history-glitch 2 months ago

Reviewers

tjund approved these changes 3 months ago
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good
The pull request has been merged as 9bd3e3a016.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#35
Loading…
There is no content yet.