Produits Entr'ouvert » Intégrations graphiques Publik

config Favicon.
Je préfère personnellement activer le mode PWA. Cela permet de laisser la main au client sur le favicon à utiliser, et permmet aux usager d'ajouter un raccourci vers la plateforme.

config "header:background": { "label": "Têtière : fond" }
je n'ai pas l'impression que cette ressource soit exploitée
Par contre le footer a une ressource en background qui n'as pas d'asset déclaré.

var $mobile-limit: $nav-mobile-limit;
Tu inverses l'héritage des variables
Si dans ton thème $mobile-limit = $nav-mobile-limit; alors fixe la valeur du breakpoint pour $mobile-limit et utilise `$min-desktop-viewport` qui fonctionne auto avec des breakpoint en `em` à la place d'un `(min-width: $nav-mobile-limit + 0.001)`

var $form-titlebar-mode: form;
Ne pas utiliser sur les nouveaux thèmes si possible.
Cette var sert à restaurer l'ancien markup qui est maintenant obsolète. (un jour cette var servira à placer le titre à cet endroit avec le bon markup, mais c'est pas encore le cas).

• Dans ta capture d'écran, tu as une image qui illustre la page. Mais du coup les liens de connexion sont masqués (passés par-dessus l'image ).
• À 989px, taille de ton $min-desktop-viewport, la nav passe sur 2 lignes et du coup les liens de connexion se retrouvent au millieu du header.

Nav
Tu utilises des images en bg pour les chevrons. Inutile. Préfère une icon Fontawesome (Font qui est déjà chargée) et qui te permet de respecter la couleur du label même si elle change en :hover et évite des bidouilles avec des `content: ' '` (<- plein d'espaces vides)

Simplifier les selecteurs pour faciliter compréhension

remplacer

div.gru-content #columns div.infos-pratiques,
div.gru-content div#rub_service div.infos-pratiques {

par

.info-pratiques {
    &.comment-field,
    &.text-cell
}

et
`div.gru-content div.shadow-button`
par
`.gru-content .cell.shadow-button`
(parce qu'il faut en finir avec les tags dans les selecteurs)

min desktop viewport

Y'en a d'autres (lines 23, 84, 94, 239 )

form title

non appliqué, selecteur h1 trop faible
text-transform: uppercase;
font-size: 1.125em;
font-weight: 500;

Attention, dans le nouveau markup, le titre est un h1 avec la class `.form-content--title`.
Cibler l'élément avec la class uniquement (jamais de tag)

nav & font-awesome

inutile :
display: inline-block;
font-weight: 100;
color: white;

Sinon, c'est un peu bizarre de donner la même taille à h1, h3 et h4.
Le titre d'un formulaire aura le même aspect que le widget "titre" et le widget "sous-titre" par ex.

Le titre 2 a l'air plus grand que le titre 1, il faut que ce soit l'inverse

Le titre 2 a l'air plus grand que le titre 1, il faut que ce soit l'inverse

Fait, mais je l'ai quand même laissé un peu plus grand pour de vrai

Ça laisse en effet une certaine incompréhension sur ce qui a en fait pu être fait.

pour garder le rendu en script

J'imagine que là-dedans il s'agit de dire "mais les h2 ne sont pas forcés en majuscules et donc apparaitront à l'œil malgré tout moins importants que les h1".

_~~

Perso, j'ai bien vu la page pointée https://www.lanester.bzh/mairie/vie-municipale/conseil-municipal/?L=1 et ok le titre principal y est riquiqui et ça justifie pour moi de ne pas suivre ça et de se permettre un titre plus grand. (sur la page en question je passe via l'inspecteur .h1-droite de font-size 1.8em à 2.8rem et ça ne me choque vraiment pas).

ok pour moi

Yes! Merci Thomas.

commit 36ca11a6d75a4dfd127bf20bac0c8751a4819903
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Feb 10 17:40:55 2022 +0100

    lanester: manage content (#61491)

Automatic expiration