Développement #47531
$max-nav-mobile-viewport & $min-nav-desktop-viewport
0%
Description
Sur le modèle de
`$max-mobile-viewport` et `$min-desktop-viewport`
introduire
`$max-nav-mobile-viewport` et `$min-nav-desktop-viewport`
facilite l'écriture des thèmes et permet de définir le breakpoint de la nav en `em`.
Files
History
Updated by Thomas Jund over 4 years ago
- File 0001-scss-introduce-max-nav-mobile-viewport-min-nav-deskt.patch 0001-scss-introduce-max-nav-mobile-viewport-min-nav-deskt.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters over 4 years ago
Il faudra ajouter ces infos dans help/fr/.
Perso (je comprends le gain à ne pas s'interroger sur +1 ou pas) mais je trouve que ça perd en lisibilité et que je me trouverai désormais à devoir chercher/copier/coller plutôt que juste taper la forme actuelle.
(voilà, je n'ai pas l'impression que ça me faciliterait la vie mais je suis bien moins qu'avant la personne qui fait les intégrations graphiques, donc ça me va d'ignorer tout ça). (mais ça me va que la validation vienne de quelqu'un d'autre du coup, quand même).
Updated by Thomas Jund over 4 years ago
je trouve que ça perd en lisibilité, […] je n'ai pas l'impression que ça me faciliterait la vie.
Au delà des avis plus/moins lisible qui sont propre à chacun et biaisé par nos habitudes, l'idée au delà de cela est aussi
- de se rapprocher de la syntaxe CSS des custom medias https://drafts.csswg.org/mediaqueries-5/#custom-mq.
- Pouvoir utiliser px ou em pour les variable *-limit.
- faciliter l'écriture des styles en 'mobile first'.
Il faudra ajouter ces infos dans help/fr/.
Je crois que je n'ai toujours pas compris à qui est destinée cette doc :(. Je ne comptais pas documenter ces variables, pensant que cela ne concernait pas le public visé.
Updated by Frédéric Péters over 4 years ago
- File Screenshot_2020-10-13 27471-theming-2 webm.png Screenshot_2020-10-13 27471-theming-2 webm.png added
Je crois que je n'ai toujours pas compris à qui est destinée cette doc :(
Ça sert à documenter les options qui peuvent être utilisées. Dans le très pratique, pour l'objectif #27471 ça sert à fournir une description à l'option, qui pourra être présentée dans l'UI (cf vidéo associée, à 00:38, capture jointe).
Maintenant, si c'est une option qui ne peut pas être utilisée (idée dont le sens peut m'avoir échappé, mais qui est bien possible en regardant le patch où jamais cette variable n'est positionnée par une intégration), ok pour ne pas avoir de documentation.
Updated by Thomas Jund about 4 years ago
J'ai pas mal utilisé le beakpoint $nav-mobile-limit pour la gestion du header pour le thème de l'isère.
Le beakpoint est en `em`.
Avant/Après ce patch.
@media (min-width: $nav-mobile-limit + 0.001) {}
VS
@media ($min-nav-desktop-viewport) {}