Development #36800
".site-wrapper" & $page-padding, une class et une variable nos marges
0%
Description
Proposition d'amélioration du core scss avec une nouvelle class : ".site-wrapper" et une nouvelle variable $page-poadding
Problèmes actuels.- En mobile, une petite marge existe entre les conteneurs et le bord de l'écran. C'est bien mais aucune variable pour gérer sa taille, aucun moyen de la passer à 0 par exemple.
- En "tablette", entre $width et $mobile-limit en fait, ces marges disparaissent, pouf.
- La valeur de $width qui permet de fixer taille max aux conteneurs est dupliqué dans chaque conteneur où elle est nécessaire.
- Graphiquement, Dans une logique d'harmonie, il est préférable d'instaurer un ratio entre ces marges et les gouttières entre les colonnes (c'est plus ou moins le cas aujourd'hui, où gouttières = marges x2 (0.7em x2), sauf pour la sidebar qui a une gouttière fixe de 20px) et un ratio avec le rythme vertical ou l'espace qui sépare 2 cellules, qui est + est égal à la marge.
- créer une class ".site-wrapper" générique qui permet lors de la création d'un nouveau bloc conteneur d'être certain d'y appliquer les bonnes marges et $width et éviter de dupliquer ces valeurs partout.
- Ajouter une variable $page-padding (parce que techniquement c'est un padding) qui permet de paramétrer la taille de marges et donc la taille des gouttières du/des layouts
- Baser la taille des gouttières sur cette nouvelle variable
Si dans la suite je reviens sur le système de grille, il sera préférable d'utiliser "$layout-gutter-size" comme variable de référence pour $page-padding. Et donc seul $layout-gutter-size sera prioritairement paramétrable dans la création de thème.
Si vous trouvez mieux que $page-padding comme nom de variable je suis preneur.
Fichiers
Demandes liées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
- Privée changé de Non à Oui
Pour ce genre type de proposition / d'évolution, qui touche à theme.html, cells.scss, nav.scss et layout.scss, je vous pousse un patch ou une branche ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
- Privée changé de Oui à Non
Mis à jour par Frédéric Péters il y a plus de 4 ans
Pour ce genre type de proposition / d'évolution, qui touche à theme.html, cells.scss, nav.scss et layout.scss, je vous pousse un patch ou une branche ?
Pousse plutôt une branche.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
Voir wip/36800_scss-page-wrapper
Mis à jour par Frédéric Péters il y a plus de 4 ans
Détails de forme avant de regarder attentivement, deux mini points : attention à ne pas laisser des espaces en fin de ligne (dans _cells.scss, il y a en derrière // In 2 cols template, add gutter et d'autres lignes), supprimer les lignes qui veulent être supprimées, plutôt que les mettre en commentaire. Et un troisième, il y a de nouvelles lignes de code ajoutées en commentaire, pas clair leur fonction.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 3 mois
- Lié à Development #85848: variable pour gérer les gouttières entre contenu et bord de l'écran ajouté
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 29 jours
- Statut changé de Nouveau à Fermé
Ce ticket peut être fermé, toutes les idées ont été intégré dans différents tickets :
- L'idée de la class ".site-wrapper" a été intégré avec les class .page-width, .header-width, .nav-width, .footer-width
- la var $mobile-width permet de limiter la largeur du contenu en mobile
- les vars $edge-gutter* permettent de définir les gouttières entre le contenu et le bord de l'écran.