finistere: add theme (#74747) #177

Merged
csechet merged 1 commits from wip/74747-finistere-integration-graphique into main 2023-02-28 16:23:45 +01:00
Owner

Après un moment à tenter de faire des choses jolies, je me suis contenté de quelque chose de très sobre, basé sur le site instit avec les nouvelles couleurs fournies, parce que je suis définitivement pas graphiste.

Après un moment à tenter de faire des choses jolies, je me suis contenté de quelque chose de très sobre, basé sur le site instit avec les nouvelles couleurs fournies, parce que je suis définitivement pas graphiste.
aberriot requested changes 2023-02-22 15:55:15 +01:00
aberriot left a comment
Owner

Relecture assez sommaire dans la mesure ou le CSS custom est très court.

Je n'ai pas repéré de problèmes de contrastes, les couleurs utilisées correspondent bien à celle de leur document.

Petite remarque : sur ta seconde capture, les toplinks masquent partiellement le logo sur mobile, peut-être qu'on peut rajouter un peu de padding pour éviter ça ?

Pour le problème de sous-menu évoqué dans un commentaire :

image

Relecture assez sommaire dans la mesure ou le CSS custom est très court. Je n'ai pas repéré de problèmes de contrastes, les couleurs utilisées correspondent bien à celle de leur document. Petite remarque : sur ta seconde capture, les toplinks masquent partiellement le logo sur mobile, peut-être qu'on peut rajouter un peu de padding pour éviter ça ? Pour le problème de sous-menu évoqué dans un commentaire : ![image](/attachments/bf0bc413-2b60-41de-ba69-ef06e65905b0)
@ -0,0 +10,4 @@
@media($min-desktop-viewport) {
.site-nav {
margin-top: -50px;
Owner

Possiblement un problème de z-index ici, chez moi un survol sur le menu ne marche pas bien car c'est le logo qui récupère plutôt que le menu. Possiblement parce que j'ai un logo aux mauvaises dimensions, mais sans logo j'ai le même pépin.

z-index: 500; sur cet élément semble résoudre le souci.

Possiblement un problème de z-index ici, chez moi un survol sur le menu ne marche pas bien car c'est le logo qui récupère plutôt que le menu. Possiblement parce que j'ai un logo aux mauvaises dimensions, mais sans logo j'ai le même pépin. `z-index: 500;` sur cet élément semble résoudre le souci.
Author
Owner

Bien vu, c'est fait.

Bien vu, c'est fait.
aberriot marked this conversation as resolved
@ -0,0 +19,4 @@
$nav-border-color: none;
$nav-height: auto !default;
$nav-menu-side: 50px !default;
$nav-menu-color: black;
Owner

Quand il y a des sous-menu, le dropdown est en fond blanc et pas très visible par rapport au reste de la page. Est-ce qu'on peut lui mettre une bordure ou quelque chose ?

Quand il y a des sous-menu, le dropdown est en fond blanc et pas très visible par rapport au reste de la page. Est-ce qu'on peut lui mettre une bordure ou quelque chose ?
Author
Owner

Ajouté une bordure.

Ajouté une bordure.
aberriot marked this conversation as resolved
@ -0,0 +33,4 @@
$title-background: transparent;
$title-color: black;
$title-font-family: Ubuntu, sans-serif;
Owner

Je ne vois pas d'autres police définie pour le reste, peut-être que ça vaudrait le coup de définir quelque chose pour éviter trop d'incohérences d'un device à l'autre ? Je vois du roboto utilisé sur https://www.finistere.fr/Le-Conseil-departemental/Le-President-du-Conseil-departemental

Je ne vois pas d'autres police définie pour le reste, peut-être que ça vaudrait le coup de définir quelque chose pour éviter trop d'incohérences d'un device à l'autre ? Je vois du roboto utilisé sur https://www.finistere.fr/Le-Conseil-departemental/Le-President-du-Conseil-departemental
Author
Owner

J'ai utilisé Ubuntu partout, je ne sais pas pourquoi je l'avais mise uniquement sur les titres.

J'ai utilisé Ubuntu partout, je ne sais pas pourquoi je l'avais mise uniquement sur les titres.
aberriot marked this conversation as resolved
aberriot approved these changes 2023-02-27 16:31:20 +01:00
aberriot left a comment
Owner

Retesté en local, tout est bon sauf les menus déroulants. La bordure les rend plus facile à discerner, mais il y a un pépin de transparence sur mon poste :

image

Je valide pour que tu puisse merger quand ça sera corrigé

Retesté en local, tout est bon sauf les menus déroulants. La bordure les rend plus facile à discerner, mais il y a un pépin de transparence sur mon poste : ![image](/attachments/de0b15ce-5345-4c4b-bac3-00d7a0160e3e) Je valide pour que tu puisse merger quand ça sera corrigé
csechet force-pushed wip/74747-finistere-integration-graphique from b2d1ea6535 to a1f0dde8a9 2023-02-28 16:23:11 +01:00 Compare
csechet merged commit a1f0dde8a9 into main 2023-02-28 16:23:45 +01:00
csechet deleted branch wip/74747-finistere-integration-graphique 2023-02-28 16:23:45 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#177
No description provided.