Project

General

Profile

Development #47531

$max-nav-mobile-viewport & $min-nav-desktop-viewport

Added by Thomas Jund over 2 years ago. Updated about 2 years ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Target version:
-
Start date:
09 October 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

#1

Updated by Thomas Jund over 2 years ago

#2

Updated by Frédéric Péters over 2 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).

#3

Updated by Thomas Jund over 2 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

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é.

#4

Updated by Frédéric Péters over 2 years ago

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.

#5

Updated by Thomas Jund about 2 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) {}

Also available in: Atom PDF