Bug #44211
Carrousel utilise le media querie de la nav ?
0%
Description
Assez étrange, l'affichage (display: block || none) des lien suivant / précédent du carrousel est conditionné par le breakpoint de la nav.
carrousel.css l.156
@media screen and (max-width: $nav-mobile-limit) {
display: none;
}
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Thomas Jund (congés, retour le 29/04)
Le problème d'utiliser le breakpoint de la nav ici est essentiellement lorsque cette valeur est modifiée dans le but de conserver une navigation en mode burger en desktop aussi (valeur très grande, 20000px, 50000px, 99999px).
On ne s'attend pas alors de perdre la possibilité de naviguer avec les flèches au sein du carrousel en desktop.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Fichier 0001-themes-carrousel-use-mobile-limit-to-display-arrows-.patch 0001-themes-carrousel-use-mobile-limit-to-display-arrows-.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Le problème d'utiliser le breakpoint de la nav ici est essentiellement lorsque cette valeur est modifiée dans le but de conserver une navigation en mode burger en desktop aussi
En soit c'est aussi ça l'erreur, il y aurait à avoir la possibilité d'un menu burger en desktop sans devoir malmener ainsi une variable sans rapport, non ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
avoir la possibilité d'un menu burger en desktop sans devoir malmener ainsi une variable sans rapport, non ?
Ça marche, mais en effet, je serais aussi pour clarifier la façon de déclarer cela.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit afe9aa8813f199956c30dbe570d72a4c2dbbbca0 (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date: Mon Mar 1 14:44:44 2021 +0100
themes: carrousel: use $mobile-limit to display arrows (#44211)
instead $nav-mobile-limit
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
themes: carrousel: use $mobile-limit to display arrows (#44211)
instead $nav-mobile-limit