Projet

Général

Profil

Bug #45965

remettre padding: 0 sur les cellules

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
19 août 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Supprimé dans #44936 mais ça produit un décalage des titres cf capture.

(erreur bissectée pour arriver à pointer #44936)


Fichiers

Révisions associées

Révision 883649be (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

scss: reset right padding for grid cells (#45965)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 3 ans

#3

Mis à jour par Thomas Jund il y a plus de 3 ans

Bug validé, mais je ne suis pas pour la réintroduction du padding 0 dans cell.scss :

Si on suit la cascade :

On ajoute un padding right à un élément avec la class .grid-* (grid.scss l.14)
Mais il ne faut pas dans le contexte grid l'appliquer sur les .cell.

2 solutions:

1/ soit on applique le padding right uniquement aux éléments de formulaires

.div[class*=grid-] {
    float: left;
        &:not(.cell) {
                padding-right: $grid-gutter;
        }
}

2/ Soit on applique le padding sur tous les éléments grid-* et on supprime ensuite pour les grid-*.cell, mais toujours au sein du fichier grid.scss, pour comprendre l'origine de ce choix.

Mais imposer un padding right au sein du fichier .grid (uniquement en contexte grid) et venir surclasser via le sélecteur pour les cellules en forçant un padding 0 dans tous les contextes et à tous les padding dans le fichier cell.scss et en plus aux éléments .block et a2-block sans aucune explication (qui ne sont jamais des .grid), n'est pas une bonne idée.

Si je l'ai enlevé, c'est parce que je pense que ce padding: 0 n'a rien à faire là.

#5

Mis à jour par Thomas Jund il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 883649bee3dfe6ea75092a7f334eff1e718a547f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Aug 21 10:19:39 2020 +0200

    scss: reset right padding for grid cells (#45965)
#7

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF