Projet

Général

Profil

Development #36800

".site-wrapper" & $page-padding, une class et une variable nos marges

Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans. Mis à jour il y a 29 jours.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
09 octobre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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.
Solution proposée:
  • 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

problem-layoutTablet-noPageMarging.png (721 ko) problem-layoutTablet-noPageMarging.png Thomas Jund (congés, retour le 29/04), 09 octobre 2019 11:40
layoutTablet-withPageMarging.png (525 ko) layoutTablet-withPageMarging.png Thomas Jund (congés, retour le 29/04), 09 octobre 2019 11:46
layoutTablet-BigPageMarging.png (482 ko) layoutTablet-BigPageMarging.png Thomas Jund (congés, retour le 29/04), 09 octobre 2019 11:52
layoutTablet-PageMarging0.png (541 ko) layoutTablet-PageMarging0.png Thomas Jund (congés, retour le 29/04), 09 octobre 2019 11:55

Demandes liées

Lié à Intégrations graphiques Publik - Development #85848: variable pour gérer les gouttières entre contenu et bord de l'écranSolution déployée18 janvier 2024

Actions

Historique

#1

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 ?

#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

  • Privée changé de Oui à Non
#3

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.

#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

Voir wip/36800_scss-page-wrapper

#5

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.

#6

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

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.

Formats disponibles : Atom PDF