Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Width grid-template in lg breakpoint #688

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

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)

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.

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
Contributor

@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