entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 10 Releases Wiki Activity

cd91: theme 2022 (#71536) #76

Merged
aberriot merged 1 commits from wip/71536-cd91 into main 2023-01-02 09:24:36 +01:00
Conversation 0 Commits 1 Files Changed 4 +231
aberriot commented 2022-12-13 17:04:07 +01:00
Owner
Copy Link

Thème inspiré de https://www.essonne.fr/, avec quelques divergences/choix (discutés avec Brice) :

Rendu

Desktop

image
image
image

Mobile

image
image
image

Thème inspiré de https://www.essonne.fr/, avec quelques divergences/choix (discutés avec Brice) : - On utilise le bleu du footer plutôt que le bleu électrique de https://www.essonne.fr/le-departement/accueil pour augmenter le contraste et avoir quelque chose de moins agressif # Rendu ## Desktop ![image](/attachments/ad97b5b9-a9a7-4955-9fbb-421ebf3ea479) ![image](/attachments/9f27c74b-af36-4b2f-89c9-b242dd2aa3f7) ![image](/attachments/8dda60ee-5b37-4902-8fb6-a37bfeb434fa) ## Mobile ![image](/attachments/2fa8ae13-7359-46e3-8cab-54c3de879fd3) ![image](/attachments/7fdd4d7d-3dde-495e-a07a-67d8d060fa87) ![image](/attachments/ff5250b6-09ac-4928-b97d-efc55cf7d5a2)
image.png
87 KiB
image.png
952 KiB
image.png
879 KiB
image.png
179 KiB
image.png
935 KiB
image.png
155 KiB
csechet reviewed 2022-12-19 17:48:03 +01:00
csechet left a comment
Owner
Copy Link

Simple et efficace.

Une remarque sur le visuel : je trouve qu'il n'y a pas beaucoup de contraste sur les chevrons et la bordure du haut dans la navigation.

Sur les variables scss qui ne sont pas utilisées, fais comme tu le sens : ça ne coute pas grand chose de les laisser pour utilisation ultérieure.

Pour le reste c'est surtout des questions sur des choses que je ne saisis pas.

Simple et efficace. Une remarque sur le visuel : je trouve qu'il n'y a pas beaucoup de contraste sur les chevrons et la bordure du haut dans la navigation. Sur les variables scss qui ne sont pas utilisées, fais comme tu le sens : ça ne coute pas grand chose de les laisser pour utilisation ultérieure. Pour le reste c'est surtout des questions sur des choses que je ne saisis pas.
static/essonne-cd91-2022/_custom.scss Outdated
@ -0,0 +13,4 @@
display: flex;
height: 6em;
flex-direction: row;
justify-content: space-between;
csechet commented 2022-12-19 17:25:46 +01:00
Owner
Copy Link

Je ne comprends pas l'utilité du space-between, puisqu'il n'y a qu'un élément dans div#top qui soit positionné (le logo, les toplinks étant en position:absolute)

Je ne comprends pas l'utilité du space-between, puisqu'il n'y a qu'un élément dans div#top qui soit positionné (le logo, les toplinks étant en position:absolute)
aberriot commented 2022-12-20 10:09:32 +01:00
Author
Owner
Copy Link

Bien vu, c'était un reliqué d'une première version discutée avec Brice en visio. Je corrige.

Bien vu, c'était un reliqué d'une première version discutée avec Brice en visio. Je corrige.
aberriot commented 2022-12-20 10:20:01 +01:00
Author
Owner
Copy Link

Je viens de pousser un fixup, je peux te laisser relire @csechet ?

Pour le manque de contraste visuel, je suis d'accord. J'ai utilisé le bleu plus clair d'https://www.essonne.fr/ dans mon dernier fixup pour les chevrons (pour la bordure, je laisse le bleu sombre, avec le bleu clair, du coup ça jure avec le reste je trouve).

Je viens de pousser un fixup, je peux te laisser relire @csechet ? Pour le manque de contraste visuel, je suis d'accord. J'ai utilisé le bleu plus clair d'https://www.essonne.fr/ dans mon dernier fixup pour les chevrons (pour la bordure, je laisse le bleu sombre, avec le bleu clair, du coup ça jure avec le reste je trouve).
aberriot marked this conversation as resolved
static/essonne-cd91-2022/_custom.scss Outdated
@ -0,0 +16,4 @@
justify-content: space-between;
align-items: center;
@media ($min-desktop-viewport) {
flex-direction: column-reverse;
csechet commented 2022-12-19 17:26:26 +01:00
Owner
Copy Link

Idem : quel intéret du column-reverse pour un seul élément.

Idem : quel intéret du column-reverse pour un seul élément.
aberriot commented 2022-12-20 10:09:42 +01:00
Author
Owner
Copy Link

(même réponse)

(même réponse)
aberriot marked this conversation as resolved
static/essonne-cd91-2022/_vars.scss Outdated
@ -0,0 +4,4 @@
$primary-color: $blue-dark;
$space-xsmall: 0.25em;
csechet commented 2022-12-19 17:45:36 +01:00
Owner
Copy Link

$space-xsmall & $space-xlarge ne sont pas utilisées.

$space-xsmall & $space-xlarge ne sont pas utilisées.
aberriot commented 2022-12-20 10:13:36 +01:00
Author
Owner
Copy Link

Effectivement, mais je préfère laisser, ça n'alourdit pas le CSS généré et ça permet de garder une cohérence et une facilité de copier-coller d'un thème à l'autre.

Effectivement, mais je préfère laisser, ça n'alourdit pas le CSS généré et ça permet de garder une cohérence et une facilité de copier-coller d'un thème à l'autre.
aberriot marked this conversation as resolved
static/essonne-cd91-2022/_vars.scss Outdated
@ -0,0 +20,4 @@
$fz-2: 24em / $base-font;
$fz-3: 21em / $base-font;
$fz-4: 18em / $base-font;
$fz-small: 14em / $base-font;
csechet commented 2022-12-19 17:45:57 +01:00
Owner
Copy Link

fz-small, fz-xsmall non utilisées.

fz-small, fz-xsmall non utilisées.
aberriot commented 2022-12-20 10:13:46 +01:00
Author
Owner
Copy Link

(même réponse)

(même réponse)
aberriot marked this conversation as resolved
csechet approved these changes 2022-12-22 12:14:34 +01:00
aberriot force-pushed wip/71536-cd91 from d64aae2740 to 0d7fa9fd76 2023-01-02 09:22:24 +01:00 Compare
aberriot merged commit 53316be14a into main 2023-01-02 09:24:36 +01:00
aberriot deleted branch wip/71536-cd91 2023-01-02 09:24:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
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/publik-base-theme#76
No description provided.
Delete Branch "wip/71536-cd91"

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?