Development #30421
Configuration d'une cellule menu de pied de page
0%
Description
Après #29129, on voudrait pouvoir configurer à coup de variables :
- la casse, la couleur, la graisse du texte des liens
- la présence d'un séparateur vertical entre les entrées
Fichiers
Demandes liées
Révisions associées
cells: horizontal separator in footer menucell (#30421)
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
- la casse, la couleur, la graisse du texte des liens
Je dirais qu'on peut s'épargner casse/graisse, infréquents, et avoir par défaut couleur = text-color($footer-background); et si jamais ça doit varier dans une intégration, que ça aille dans le _custom.scss.
(parce que faire footer .menucell a { ... } est facile).
- la présence d'un séparateur vertical entre les entrées
Ça, oui, pour ne pas demander à l'auteur des CSS de jouer avec des :last-child que sais-je.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Fichier 0001-layout-restrict-footer-background-to-background-colo.patch 0001-layout-restrict-footer-background-to-background-colo.patch ajouté
- Fichier 0002-cells-horizontal-separator-in-footer-menucell-30421.patch 0002-cells-horizontal-separator-in-footer-menucell-30421.patch ajouté
- Fichier Screenshot-2019-2-6 Portail - Accueil.png Screenshot-2019-2-6 Portail - Accueil.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Premier patch un peu de ménage parce que deux thèmes utilisent $footer-background
pour plus que de la couleur, ce qui faisait planter text-color($footer-background);
Ensuite en plus du séparateur, j'adapte le padding parce que sinon on a 1rem dans toutes les directions qui vient de _cells.scss et qui fait des séparateurs nettement trop grands par rapport au texte à mon sens.
Capture d'écran sans aucun artifice de #30412 à part le centrage et la largeur du pied de page, juste ce patch, une cellule texte grid-1-6 pour l'image et une cellule menu grid-5-6.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Lié à Bug #30412: cd13 : marges entre texte et séparateur vertical ne s'adpate pas à la longueur du texte ajouté
Mis à jour par Frédéric Péters il y a environ 5 ans
Tu as testé le rendu rochefort/urville-nacqueville ?
border-right: 2px solid;
Je me dis que la valeur de cette propriété pourrait venir de $footer-menucell-separator, plutôt qu'y mettre un booléen.
Ainsi on y aurait none par défaut mais on pourrait faire $footer-menucell-separator: 1px solid white; si jamais on voulait une bordure d'un pixel, plutôt.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Fichier 0002-cells-horizontal-separator-in-footer-menucell-30421.patch 0002-cells-horizontal-separator-in-footer-menucell-30421.patch ajouté
Frédéric Péters a écrit :
Tu as testé le rendu rochefort/urville-nacqueville ?
Ok sur Rochefort et urville-nacqueville n'est plus utilisé (https://portail-usager-urville-nacqueville-hague.lahague.com/manage/).
[...]
Je me dis que la valeur de cette propriété pourrait venir de $footer-menucell-separator, plutôt qu'y mettre un booléen.
Oui c'est plus malin, voilà.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Statut changé de Solution proposée à En cours
Emmanuel Cazenave a écrit :
Frédéric Péters a écrit :
Tu as testé le rendu rochefort/urville-nacqueville ?
Ok sur Rochefort.
En fait non je me suis emmêlé les pinceaux.
Mis à jour par Frédéric Péters il y a environ 5 ans
Ok et reste ausis à ajouter dans help/fr/misc-scss.page les infos sur ce paramètre.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Fichier 0001-layout-restrict-footer-background-to-background-colo.patch 0001-layout-restrict-footer-background-to-background-colo.patch ajouté
- Fichier 0002-cells-horizontal-separator-in-footer-menucell-30421.patch 0002-cells-horizontal-separator-in-footer-menucell-30421.patch ajouté
- Statut changé de En cours à Solution proposée
Ok pour Rochefort et même technique pour urville-nacqueville dès fois qu'il ressuscite.
Documenté dans la section 'cellules', les autres entrées de type footer* sont dans les 'généralités' mais ça me semble bien trop spécifique pour cette section.
Mis à jour par Frédéric Péters il y a environ 5 ans
Ok; je pense que le @if $footer-menucell-separator != none n'est même pas nécessaire, le "none" est une valeur ok pour border-right. (et c'est plutôt logique que le padding ne soit pas lié à la présence d'un séparateur).
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Fichier 0002-cells-horizontal-separator-in-footer-menucell-30421.patch 0002-cells-horizontal-separator-in-footer-menucell-30421.patch ajouté
Sans le if avec padding: 0.5rem 1rem;
pour tout le monde (pas sûr si tu voulais ça ou au contraire rester sur le padding par défaut).
Mis à jour par Frédéric Péters il y a environ 5 ans
Je pensais que tu voulais le padding parce que sinon les barres étaient trop hautes.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Ok, très bien ainsi, j'avais uniquement réagi au commentaire en y lisant trop vite "sans le padding...".
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9b13d4446c01ad0e58109c42dfea4380b9c7613d Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Wed Feb 6 15:57:32 2019 +0100 cells: horizontal separator in footer menucell (#30421)
commit 18b1546f27d1be80920904ad5d0665d70da45109 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Wed Feb 6 15:56:32 2019 +0100 layout: restrict $footer-background to background-color (#30421)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
layout: restrict $footer-background to background-color (#30421)