entrouvert
/
gadjo
15
0
Fork 0
Code Issues Pull Requests 3 Releases Activity

colors chart (via css vars) #20

Merged
tjund merged 1 commits from wip/71879-colors-css-vars into main 2023-05-17 15:12:25 +02:00
Conversation 2 Commits 1 Files Changed 2 +68 -33
tjund commented 2023-05-10 16:35:00 +02:00
Owner
Copy Link
No description provided.
tjund added 1 commit 2023-05-10 16:35:01 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
54cf20476b dark mode tests
tjund added 1 commit 2023-05-10 18:13:13 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
718f677999 isolate colors in new file and import chrono colors
tjund commented 2023-05-10 18:16:49 +02:00
Author
Owner
Copy Link

Je propose un commit en plus qui :

  • Isole le nuancier dans un fichier
  • ajoute le nuancier de chrono dans l'idée de le généraliser (parce qu'il est dejà a11y friendly) et utilisé.

(Avec l'idée dans un second temps de reprendre les différents composants avec ce nuancier).

Je propose un commit en plus qui : * Isole le nuancier dans un fichier * ajoute le nuancier de chrono dans l'idée de le généraliser (parce qu'il est dejà a11y friendly) et utilisé. (Avec l'idée dans un second temps de reprendre les différents composants avec ce nuancier).
tjund added 1 commit 2023-05-11 11:17:11 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
f4f22e9cd7 remove (prefers-color-scheme: dark) and --white
tjund commented 2023-05-11 11:19:55 +02:00
Author
Owner
Copy Link

Suppression des @media (prefers-color-scheme: dark) et de la var --white que je trouve mal nommée et relative au color-scheme dark.

Suppression des `@media (prefers-color-scheme: dark)` et de la var --white que je trouve mal nommée et relative au color-scheme dark.
tjund added 2 commits 2023-05-15 17:47:27 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
ee93fbb0d1 remove @debug
tjund force-pushed wip/71879-colors-css-vars from ee93fbb0d1 to f75b5a520e 2023-05-15 18:00:01 +02:00 Compare
tjund changed title from WIP: colors chart (via css vars) to colors chart (via css vars) 2023-05-16 15:36:22 +02:00
tjund requested review from aberriot 2023-05-16 15:36:44 +02:00
aberriot approved these changes 2023-05-17 09:11:05 +02:00
aberriot left a comment
Owner
Copy Link

Testé en local, pas de régression constatée, à part mon micro commentaire sur la variable vide :)

Testé en local, pas de régression constatée, à part mon micro commentaire sur la variable vide :)
gadjo/static/css/gadjo.scss Outdated
@ -70,3 +61,3 @@
}
&.disabled {
color: #aaa;
color: var();
aberriot commented 2023-05-17 09:08:34 +02:00
Owner
Copy Link

Une var vide ici, je pense qu'il manque quelque chose ?

Une var vide ici, je pense qu'il manque quelque chose ?
tjund commented 2023-05-17 15:09:03 +02:00
Author
Owner
Copy Link

Merci pour la vigilance. ligne restaurée.

Merci pour la vigilance. ligne restaurée.
aberriot marked this conversation as resolved
tjund added 1 commit 2023-05-17 15:08:13 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
7fa84c3515 css: create colors chart file (#71879)
aberriot approved these changes 2023-05-17 15:10:48 +02:00
tjund merged commit a2b2605df0 into main 2023-05-17 15:12:25 +02:00
tjund deleted branch wip/71879-colors-css-vars 2023-05-17 15:12:25 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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/gadjo#20
No description provided.
Delete Branch "wip/71879-colors-css-vars"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?