Development #67642
Ajouter une variable scss carrousel-navigation-color
0%
Description
Qui pourrait être utile pour modifier à la fois la couleur des flêches de navigation et de la puce.
Fichiers
Révisions associées
Historique
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier
0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patchsupprimé
Mis à jour par Corentin Séchet il y a presque 2 ans
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patch 0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patch ajouté
Aucun besoin de modifier les thèmes existants en fait.
Mis à jour par A. Berriot il y a presque 2 ans
Corentin Séchet a écrit :
Aucun besoin de modifier les thèmes existants en fait.
Ça me parait okay, juste une question, je vois un passage de
color: white
à color: $carrousel-arrows-color;
dans static/includes/_carrousel.scss
Il ne risque pas d'y avoir d'effet de bord sur l'existant ?
Mis à jour par A. Berriot il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Jund il y a presque 2 ans
je ne vois pas de réponse ?
Ça me parait okay, juste une question, je vois un passage de
color: white à color: $carrousel-arrows-color;
Mis à jour par A. Berriot il y a presque 2 ans
Thomas Jund a écrit :
je ne vois pas de réponse ?
j'avais passé en validé de manière optimiste, vu que ça me semblait un point de détail, histoire de pas bloquer le merge si ça a déjà été testé ou envisagé
Mis à jour par Thomas Jund il y a presque 2 ans
Il ne risque pas d'y avoir d'effet de bord sur l'existant ?
C'est aussi mon sentiment.
Je me suis permis de rétrograder le ticket de "solution validée" à "en cours" parce que je ne vois de réponse de Corentin à ta question.
(le `color: white` l.92 pose la couleur du texte de la slide)
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patch 0001-scss-add-carrousel-navigation-color-and-carrousel-ar.patch ajouté
- Statut changé de En cours à Solution proposée
(le `color: white` l.92 pose la couleur du texte de la slide)
C'était une bourde de ma part effectivement.
color: white à color: $carrousel-arrows-color;
dans static/includes/_carrousel.scss
Il ne risque pas d'y avoir d'effet de bord sur l'existant ?
Si en effet, la deuxième modification est l'objet du ticket. J'ai fait le tour des thèmes qui utilisent le carrousel : soit $button-color est à blanc, donc pas besoin de redéfinir $carrousel-arrows-color dans le _vars.scss, soit les flèches ne sont pas affichées ($carrousel-arrows: none).
Mis à jour par Thomas Jund il y a plus d'un an
Quelques propoistions côté doc :
Couleur des items de navigation (disque et flêches gauche / droite)
Simplifier par
Couleur des éléments de navigation (disques et flèches)
Couleur des flèches (gauche / droite) permettant de passer d'un élément à l'autre
Simplifier par
Couleur des flèches de navigation
Mis à jour par Corentin Séchet il y a plus d'un an
Mis à jour par Thomas Jund il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Corentin Séchet il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 6344f898cad6d74d56dae69c004a2590ef70b101 Author: Corentin Séchet <csechet@entrouvert.com> Date: Fri Jul 22 12:15:25 2022 +0200 scss: add carrousel-navigation-color and carrousel-arrows-color variables (#67642)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
scss: add carrousel-navigation-color and carrousel-arrows-color variables (#67642)