Projet

Général

Profil

Development #47531

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

Ajouté par Thomas Jund il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Solution proposée
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
09 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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


Fichiers

Historique

#1

Mis à jour par Thomas Jund il y a plus de 3 ans

#2

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

Mis à jour par Thomas Jund il y a plus de 3 ans

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

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

Mis à jour par Thomas Jund il y a plus de 3 ans

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) {}

Formats disponibles : Atom PDF