Project

General

Profile

Development #62142

barre de défilement horizontal quand la grille flex est utilisée sur le contenu

Added by Frédéric Péters over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
24 February 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files


Related issues

Related to Combo - Development #62072: disposition en grille du contenu d'un placeholderFermé22 February 2022

Actions
Related to w.c.s. - Development #62144: poser overflow-x: auto; sur div#main-content, spécifique wcsFermé24 February 2022

Actions
Related to Combo - Bug #62182: manager, ne pas laisser la barre latérale être poussée par du contenu trop largeFermé25 February 2022

Actions
Related to Gadjo - Development #62249: remettre les règles overflow sur le contenuFermé28 February 2022

Actions

Associated revisions

Revision 38338670 (diff)
Added by Frédéric Péters over 2 years ago

misc: remove content overflow rules, only appopriate for wcs (#62142)

History

#1

Updated by Thomas Jund over 2 years ago

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;
}

#2

Updated by Frédéric Péters over 2 years ago

#3

Updated by Frédéric Péters over 2 years ago

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.

#4

Updated by Frédéric Péters over 2 years ago

  • Related to Development #62144: poser overflow-x: auto; sur div#main-content, spécifique wcs added
#5

Updated by Frédéric Péters over 2 years ago

#6

Updated by Thomas Jund over 2 years ago

  • Status changed from Solution proposée to Solution validée
#7

Updated by Frédéric Péters over 2 years ago

  • Status changed from Solution validée to Résolu (à déployer)
  • Assignee set to 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)
#8

Updated by Frédéric Péters over 2 years ago

Ç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.

#9

Updated by Frédéric Péters over 2 years ago

  • Related to Bug #62182: manager, ne pas laisser la barre latérale être poussée par du contenu trop large added
#10

Updated by Frédéric Péters over 2 years ago

Ç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.

#11

Updated by Frédéric Péters over 2 years ago

#12

Updated by Transition automatique over 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#13

Updated by Transition automatique about 2 years ago

Automatic expiration

Also available in: Atom PDF