Produits Entr'ouvert » Intégrations graphiques Publik

J'ai pushé ça sur la branche :

• Fixé une typo (Entête -> En-tête)
• Supprimé une propriété commentée
• Le placeholder par défaut était toujours affiché même s'il en avait un de défini (soit utiliser le selecteur cell + ô-header pour le cacher, soit ajouter la classe default-header au header par défaut, j'ai fait la seconde solution)
• Je ne sais pas si c'était voulu, mais j'ai mis un background transparent pour les text-cell du header, le background blanc masquait le "ô"
• Supprimé une règle media vide
• Tu utilisais salmon à la place de $salmon pour la nav-active-color dans _vars.scss, j'imagine que c'était une typo
• J'ai changé les couleurs de la navigation PWA en mettant celles de la nav standard, c'était blanc sur blanc.

Et autres remarques / questions :

• Pourquoi utiliser une variable hobo toulouse_site_title et pas site_title qui existe déjà ?
• Pour les !important, on m'a dit que c'était mal, mais tant que c'est pas dans le core, j'ai pas trop d'argument contre.
• Mon avis, mais je suis pas ergonome : je trouve les liens dans les sous-menus très petits et difficiles à lire

Merci Corentin, super retour.

Fixé une typo (Entête -> En-tête)
Supprimé une propriété commentée
Supprimé une règle media vide

Thanks

Le placeholder par défaut était toujours affiché même s'il en avait un de défini (soit utiliser le selecteur cell + ô-header pour le cacher, soit ajouter la classe default-header au header par défaut, j'ai fait la seconde solution)
Je ne sais pas si c'était voulu, mais j'ai mis un background transparent pour les text-cell du header, le background blanc masquait le "ô"

Bien vu, j'ai écris un patch similaire ce matin.

Tu utilisais salmon à la place de $salmon pour la nav-active-color dans _vars.scss, j'imagine que c'était une typo

Oui, pas remarqué car 'salmon' est un keyword couleur

J'ai changé les couleurs de la navigation PWA en mettant celles de la nav standard, c'était blanc sur blanc.

OK, je garde, d'après la charte, il n'y a pas prévu de nav PWA, ça passera en second temps si le besoin se manifeste.

Pour les !important, on m'a dit que c'était mal, mais tant que c'est pas dans le core, j'ai pas trop d'argument contre.

Voilà, éviter au max dans le core (ou alors sur des class utilitaires). Ne me dérange pas dans les thèmes si cela permet de simplifier le CSS.
Par exemple

%button {
    box-shadow: none !important;
}

C'est pour éviter d'écrire

%button {
    box-shadow: none;
    &:hover {
        box-shadow: none;
    }
}

ou

%button {
    &, &:hover {
        box-shadow: none;
    }
}

Pourquoi utiliser une variable hobo toulouse_site_title et pas site_title qui existe déjà ?

Têtière:logo est chargé en BG image, n'apporte aucune information SEO ou a11y.

Dans notre cas, en utilisant page_title pour l'information 'Démarches en ligne" voulu par la maquette, le titre du site ne portera aucune information relative à "Toulouse mairie - metropole", ce qui est assez dangereux.
J'ai donc choisi d'utiliser "site_title" pour entrer l'information "Toulouse mairie - metropole", en "sr-only" (une sorte d'attr alt) et d'ajouter un titre visible "Démarches en ligne" à travers une variable hobo.

Utiliser site_title uniquement et visible est valable si son contenu apporte assez d'information pour identifier la collectivité et la plateforme.
Lorsque site_title est masqué, le problème ne se pose pas, puisqu'on peut y insérer toute l'information necessaire.

Mon avis, mais je suis pas ergonome : je trouve les liens dans les sous-menus très petits et difficiles à lire

Pas testé. La charte ne prévoit pas l'utilisation de sous-menus

Automatic expiration