css: extract color from background variable in contexts requiring it (#78673) #319

fpeters · 2023-06-18T13:27:27+02:00

fpeters commented

2023-06-18 13:27:27 +02:00

No description provided.

fpeters added 1 commit 2023-06-18 13:27:27 +02:00

gitea/publik-base-theme/pipeline/head This commit looks good Details

8b9ac000d1 css: extract color from background variable in contexts requiring it (#78673 )

tjund commented

2023-06-19 14:58:24 +02:00

Dans la doc, il est noté par exemple : "$form-background: Couleur de fond d'une démarche."
Cela demande une mise à jour de la doc ?

Au delà, si on autorise toutes les valeurs possibles de CSS background, il y a des cas où extract-color() ne saura trouver de couleur et retournera sonc transparent, sans alerte particulière, je ne sais pas si c'est bien.

Et sur $table-wrapper-background il faut absolument une couleur unie (et un gradient par exemple) pour le fonctionnement de l'affichage des shadows

Dans la doc, il est noté par exemple : "$form-background: Couleur de fond d'une démarche." Cela demande une mise à jour de la doc ? Au delà, si on autorise toutes les valeurs possibles de CSS background, il y a des cas où extract-color() ne saura trouver de couleur et retournera sonc transparent, sans alerte particulière, je ne sais pas si c'est bien. Et sur $table-wrapper-background il faut absolument une couleur unie (et un gradient par exemple) pour le fonctionnement de l'affichage des shadows

fpeters commented

2023-06-28 08:44:29 +02:00

Dans la doc, il est noté par exemple : "$form-background: Couleur de fond d'une démarche."
Cela demande une mise à jour de la doc ?

Je me suis posé la question et je pense que non, ça reste plutôt bien qu'elle concerne le cas très commun.

Au delà, si on autorise toutes les valeurs possibles de CSS background, il y a des cas où extract-color() ne saura trouver de couleur et retournera sonc transparent, sans alerte particulière, je ne sais pas si c'est bien.

Ça n'est certainement pas idéal mais c'est mieux que la production de code invalide qui peut aujourd'hui arriver, et pour lequel on n'a pas non plus d'alerte.

exemple saint-laurent-du-var qui contient actuellement :

  box-shadow: 0 0 0px 5px #eee url(img/background.jpg); }

avec le patch appliqué ça donnera box-shadow: 0 0 0px 5px #eee; qui est correct.

Et sur $table-wrapper-background il faut absolument une couleur unie (et un gradient par exemple) pour le fonctionnement de l'affichage des shadows

Je ne comprends pas cette remarque, sur les tableaux le patch fait :

-		linear-gradient(to right, $table-wrapper-background 30%, transparent),
-		linear-gradient(to right, transparent, $table-wrapper-background 70%) 0 100%,
+		linear-gradient(to right, extract-color($table-wrapper-background) 30%, transparent),
+		linear-gradient(to right, transparent, extract-color($table-wrapper-background) 70%) 0 100%,

donc il me semble plutôt améliorer la situation, faire en sorte qu'il y ait bien une couleur retournée, plutôt qu'une propriété plus longue.

> Dans la doc, il est noté par exemple : "$form-background: Couleur de fond d'une démarche." > Cela demande une mise à jour de la doc ? Je me suis posé la question et je pense que non, ça reste plutôt bien qu'elle concerne le cas très commun. > Au delà, si on autorise toutes les valeurs possibles de CSS background, il y a des cas où extract-color() ne saura trouver de couleur et retournera sonc transparent, sans alerte particulière, je ne sais pas si c'est bien. Ça n'est certainement pas idéal mais c'est mieux que la production de code invalide qui peut aujourd'hui arriver, et pour lequel on n'a pas non plus d'alerte. exemple saint-laurent-du-var qui contient actuellement : ``` box-shadow: 0 0 0px 5px #eee url(img/background.jpg); } ``` avec le patch appliqué ça donnera `box-shadow: 0 0 0px 5px #eee;` qui est correct. > Et sur $table-wrapper-background il faut absolument une couleur unie (et un gradient par exemple) pour le fonctionnement de l'affichage des shadows Je ne comprends pas cette remarque, sur les tableaux le patch fait : ``` - linear-gradient(to right, $table-wrapper-background 30%, transparent), - linear-gradient(to right, transparent, $table-wrapper-background 70%) 0 100%, + linear-gradient(to right, extract-color($table-wrapper-background) 30%, transparent), + linear-gradient(to right, transparent, extract-color($table-wrapper-background) 70%) 0 100%, ``` donc il me semble plutôt améliorer la situation, faire en sorte qu'il y ait bien une couleur retournée, plutôt qu'une propriété plus longue.

csechet approved these changes 2023-09-19 17:25:41 +02:00

tjund requested changes 2023-09-20 11:09:11 +02:00

tjund left a comment

Merci Corentin d'avoir remonté ce ticket, complètement oublié.
J'ai juste poussé / proposé un commit qui propose de poser extract-color directement sur $table-wrapper-background qui doit être une couleur unie.

Merci Corentin d'avoir remonté ce ticket, complètement oublié. J'ai juste poussé / proposé un commit qui propose de poser extract-color directement sur $table-wrapper-background qui doit être une couleur unie.

tjund added 1 commit 2023-09-20 11:11:01 +02:00

gitea/publik-base-theme/pipeline/head This commit looks good Details

a5ccec546e $table-wrapper-color need to be a solid color

csechet approved these changes 2023-11-10 14:39:58 +01:00

fpeters force-pushed wip/78673-cell-background--extract-color from a5ccec546e to c821a41fa9

2023-11-24 09:06:02 +01:00

Compare

fpeters merged commit c821a41fa9 into main

2023-11-24 09:11:11 +01:00

fpeters deleted branch wip/78673-cell-background--extract-color

2023-11-24 09:11:11 +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#319