Development #54416
permettre la personnalisation de la couleur de fond du séparateur "OU"
0%
Description
Ajourd'hui c'est "white". On devrait permettre la personnalisation de la couleur.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Fichier 0001-misc-allow-custom-background-for-OU-separator-54416.patch 0001-misc-allow-custom-background-for-OU-separator-54416.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Lié à Development #54246: rapatrier le thème GLC ajouté
Mis à jour par Thomas Jund il y a presque 3 ans
Mis à jour par Thomas Jund il y a presque 3 ans
(Merde message de perdu, je recommence)
le "OU" est composé d'un filet (border-left du block) qu'il vient masqué à l'aide d'un background au "OU".
Le background n'est qu'un masque, un pansement. Et je suis pas trop fan d'ajouter une varaible pour customiser un cache.
Il faudrait que le "OU" n'ai tout simplement pas de background. Et donc construire le filet avec autre chose que le border du block.
J'ai fait une tentative avec gradient (voir patch)
Ce procedé nous premettrait même de faire cohexister le separateur avec $cell-background
Si pas chaud pour mon patch, sur ton idée, j'ajouterais $body-background comme valeur par défaut au cache.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Fichier 0001-scss-display-OR-separator-in-mobile-view-54416.patch 0001-scss-display-OR-separator-in-mobile-view-54416.patch ajouté
Je suis d'accord avec ton raisonnement et ton patch va dans le bon sens.
Il reste qu'on a fait le choix de ne pas afficher de "OU" en vue mobile, mais on a des intégrations qui l'affichent à l'horizontale en mobile: genre GLC, Arles.
Je propose de complèter ton patch patch pour afficher le séparateur en mobile (je ne suis pas très l'aise avec linear-gradient).
Mis à jour par Thomas Jund il y a presque 3 ans
J'ai essayé de simplifier le code. Surtout la définition des gradients.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Fichier 0001-scss-use-gradient-instead-border-to-generate-OR-sepa.patch 0001-scss-use-gradient-instead-border-to-generate-OR-sepa.patch ajouté
Voilà le tout en un seul patch.
Pour moi c'est ok.
Mis à jour par Thomas Jund il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
commit 4588675bb9786fb3abee5c90b14e34ffbf119766 (origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date: Mon May 31 15:53:49 2021 +0200
scss: use gradient instead border to generate OR separator line (#54416)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 3 ans
- Lié à Development #54507: arles-2020: supprimer les séparateurs "OU" personnalisés ajouté
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: use gradient instead border to generate OR separator line (#54416)