finistere: add theme (#74747) #177
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/74747-finistere-integration-graphique"
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?
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.
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 :
@ -0,0 +10,4 @@
@media($min-desktop-viewport) {
.site-nav {
margin-top: -50px;
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.Bien vu, c'est fait.
@ -0,0 +19,4 @@
$nav-border-color: none;
$nav-height: auto !default;
$nav-menu-side: 50px !default;
$nav-menu-color: black;
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 ?
Ajouté une bordure.
@ -0,0 +33,4 @@
$title-background: transparent;
$title-color: black;
$title-font-family: Ubuntu, sans-serif;
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
J'ai utilisé Ubuntu partout, je ne sais pas pourquoi je l'avais mise uniquement sur les titres.
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 :
Je valide pour que tu puisse merger quand ça sera corrigé
b2d1ea6535
toa1f0dde8a9