Bug #38821
links-list: modifier séparateur entre liens
0%
Description
Sur le thème Meuse-cd55 j'ai repéré des séparateurs entre le lien et sa description dans les cellules.
Voir capture
Ces séparateurs sont normalement "masqués" (dans le core) à l'aide d'un border-top ajouté à la description qui a la même couleur que le background de la cellule et qui vient se superposer à la bordure du lien.
Et cette astuce ne fonctionne plus quand:
$cell-background: transparent;
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
- Fichier separateur-entre-lien-description-patch.png separateur-entre-lien-description-patch.png ajouté
- Fichier 0001-scss-links-list-use-border-top-instead-of-bottom-as-.patch 0001-scss-links-list-use-border-top-instead-of-bottom-as-.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Thomas Jund (congés, retour le 29/04)
- Patch proposed changé de Non à Oui
Je propose de simplifier le code CSS et supprimer l'astuce de la double bordure superposée en utilisant comme séparateur le border-top à la place du border-bottom.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Ça casse le rendu des faux blocs de liens créés via l'utilisation de la classe no-bottom-margin (ajoutée dans #16147), il y a un exemple d'utilisation https://portail-mel.test.entrouvert.org/ pour le lien "demande de logement social"; à noter ici deux choses : 1/ ça a été ajouté pour faire des blocs de liens et donc on pourrait oublier cet usage une fois qu'on aura la cellule liste de liens, 2/ la classe me semble aussi utilisée à quantité d'autres endroits qui ne seraient pas touchés.
(aussi le patch ajoute une tabulation en fin de ligne, ce que git signale ("1 ligne a ajouté des erreurs d'espace.").)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
- Fichier 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch ajouté
Autre solution qui permet de prendre en compte les cellules "collées" est d'utiliser non pas le border-bottom du lien, mais de l'item comme séparateur.
div.links-list ul {
& > li {
+ border-bottom: 1px solid $cell-entry-border-color;
> a {
- border-bottom: 1px solid $cell-entry-border-color;
J'ai relevé que peu de thèmes avaient modifié le comportement des séparateurs : fontenay-sous-bois et Arles.
Ces thèmes sont corrigés dans ce patch.
Mis à jour par Nicolas Roche il y a plus de 4 ans
Je n'ai pas assez de connaissances CSS et de l'historique de Publik pour valider ici (notamment vu la remarque de Fred).
Je dit juste en passant (testé via #39091),
ce patch place bien le trait de démarcation au dessous d'un éventuel texte qui serait ajouté sous les liens,
y compris avec les thème d'Arles et de Fontenanay-sous-bois.
Mis à jour par Frédéric Péters il y a environ 4 ans
Petit truc à résoudre, le patch pose problème sur une cellule menu qui contient un second niveau, cf captures.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Fichier 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch ajouté
Patch avec ajout d'un séparateur en border-top pour les menucell
-div.menucell ul ul a {
- padding-left: 3em;
- font-size: 90%;
-}
+div.menucell li li {
+ border-bottom: none;
+ border-top: 1px solid $cell-entry-border-color;
+ a {
+ padding-left: 3em;
+ font-size: 90%;
+ }
+}
Mis à jour par Nicolas Roche il y a environ 4 ans
stp, est-ce que tu peux rebaser ton patch, je n'arrive plus à l'appliquer ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Fichier 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch ajouté
Patch rebasé.
Le nesting et les indentations de sass ne sont pas pratique avec git.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Fichier 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch 0001-scss-links-list-use-li-border-bottom-as-item-separat.patch ajouté
Un patch plus clean, sans réindentation.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
git am me signale qu'il y a un espace en trop à la fin de ligne 344 de static/includes/_cells.scss
A par ça pour moi c'est bon : le dernier patch corrige également le problème sur les cellules de second niveau de menu.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
commit b013e77208dd47e2468cccfe0146642b43ba2749 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date: Tue Jan 7 14:55:25 2020 +0100
scss: links-list: use li border-bottom as item separator (#38821)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: links-list: use li border-bottom as item separator (#38821)