Development #62142
barre de défilement horizontal quand la grille flex est utilisée sur le contenu
0%
Description
Dans #62072#note-1 j'écris :
Aussi, ça donne une barre de scroll horizontal, il y a quelque chose qui déborde, ticket à faire côté gadjo.
Voilà ce ticket, endroit approprié pour en parler.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans
Aussi, ça donne une barre de scroll horizontal, il y a quelque chose qui déborde, ticket à faire côté gadjo.
les gouttières sont gérés via margins negatifs (en attendant de pouvoir utiliser la prop "gap" pour flexbox).
Cela ne génère habituellement pas de scrollbar sauf si un parent a déclaré un overflow-x en auto ou en scroll.
Et c'est le cas sur div#main-content dans gadjo.scss
div#main-content {
-ms-flex: 1 1 auto;
flex: 1 1 auto;
width: 100%;
margin: 1rem;
min-height: 90vh;
position: relative;
overflow-x: auto;
overflow-y: hidden;
box-sizing: border-box;
@media screen and (max-width: $mobile-limit) {
border-width: 0 0 1px 0;
margin: 0;
padding: 1rem;
}
}
Ajouté via #5304 sans trop comprendre pourquoi.
Si ça n'a plus de raison d'être, il faut supprimer le `overflow-x`.
Une autre solution, interessant aussi est d'alouer une balise specifique au placeholder qui peux alors être stylé sans entrer en conflit avec les styles de fx-grid et y ajouter un overflow specifique:
<div class="combo-placeholder combo-placeholder-PlaceholderName">
<div class="fx-grid">
<cell>
</div>
</div>
.combo-placeholder {
overflow: hidden;
}
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Development #62072: disposition en grille du contenu d'un placeholder ajouté
Mis à jour par Frédéric Péters il y a environ 2 ans
Si ça n'a plus de raison d'être, il faut supprimer le `overflow-x`.
Le ticket pointe que ça vient de l'incorporation de styles de w.c.s., ça vient des pages de tableau de traitement de w.c.s. qui peuvent être larges et doivent être scrollables, plutôt que pousser la barre latérale hors de l'écran.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Development #62144: poser overflow-x: auto; sur div#main-content, spécifique wcs ajouté
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-misc-remove-content-overflow-rules-only-appopriate-f.patch 0001-misc-remove-content-overflow-rules-only-appopriate-f.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Frédéric Péters
commit 38338670a02cfb8c3e6cb419e39e8dcf7278e0ac
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Feb 24 15:27:07 2022 +0100
misc: remove content overflow rules, only appopriate for wcs (#62142)
Mis à jour par Frédéric Péters il y a environ 2 ans
Ça avait aussi un bénéfice côté combo, quand il y a un <select> avec une longue option la barre latérale est désormais repoussée. J'espère qu'on ne trouvera pas trop d'autres situations.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Bug #62182: manager, ne pas laisser la barre latérale être poussée par du contenu trop large ajouté
Mis à jour par Frédéric Péters il y a environ 2 ans
Ça pète aussi authentic, vu sur la page d'import de comptes.
Je vais donc proposer d'arrêter les frais ici et annuler ce patch.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Development #62249: remettre les règles overflow sur le contenu ajouté
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: remove content overflow rules, only appopriate for wcs (#62142)