Skip to content

Width grid-template in lg breakpoint#688

Open
xentenza wants to merge 1 commit intodevfrom
styleguide/lg-width
Open

Width grid-template in lg breakpoint#688
xentenza wants to merge 1 commit intodevfrom
styleguide/lg-width

Conversation

@xentenza
Copy link
Collaborator

@xentenza xentenza commented Nov 8, 2023

Dans les écrans entre 992 et 1200px (iPad pro par exemple), la largeur de grid-column > content-width (les textes, par exemple) est plus grande que celle dans les écrans de plus grande taille.

Pour une largeur standard de 570px, nous sautons à 640px.
Le nombre de caractères par ligne passe des 70 recommandés à plus de 80 caractères par ligne.

Proposal for a new "lg" width
@xentenza
Copy link
Collaborator Author

xentenza commented Nov 8, 2023

@alinekeller Je vais voir si je vois des effets de bord; en attendant, je te laisse voir si c'est justifié ou non :-)
(remarque reçue de la part d'un des graphistes Mediacom)

@github-actions
Copy link

github-actions bot commented Nov 8, 2023

Unit Test Results

    1 files      1 suites   0s ⏱️
265 tests 262 ✔️ 0 💤 0  3 🔥
265 runs  259 ✔️ 0 💤 3  3 🔥

For more details on these errors, see this check.

Results for commit 9373d3d.

@github-actions
Copy link

github-actions bot commented Nov 8, 2023

🔎 Download the Backstop report for this pull request (link valid for 90 days):

@alinekeller
Copy link
Collaborator

@alinekeller Je vais voir si je vois des effets de bord; en attendant, je te laisse voir si c'est justifié ou non :-) (remarque reçue de la part d'un des graphistes Mediacom)

@xentenza Écoute je pense que jamais aucun internaute n'a remarqué ça, mais si ça peut faire plaisir aux graphistes… 😅

À ce moment là j'uniformiserais vraiment tout: entre 768 et 991px la largeur est aussi supérieure à 570px (en l'occurence 600px). Autant tout mettre à 570px. Il faudrait donc aussi modifier la ligne 37 et remplacer [content-start] #{col-width(md, 10)} par [content-start] #{col-width(md, 9.5)}.

Je ne comprends pas bien comment Bootstrap calcule ces colonnes, le code compilé est redondant car identique pour chaque breakpoint. J'ai fait un test rapide et j'ai l'impression qu'on devrait pouvoir retirer les lignes 42 à 56 (breakpoints lg et xl), mais je ne suis pas sûre que ça ne crée pas de problème ailleurs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants