Development #44936
Development #44934: Plan amélioration layout des templates
supprimer padding:0 inutile sur les cellules
0%
Description
Dans _cell.scss; ligne 87, un padding:0 est attribué aux cellules.
Je propose sa suppression parce que :
- aucun padding n'est déclaré en amont (il n'y a pas de reset à faire)
- padding a une valeur de 0 par défaut sur les div et les li
Pour moi cette déclaration est inutile.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund il y a plus de 3 ans
- Fichier 0001-css-remove-useless-padding-0-on-cell-44936.patch 0001-css-remove-useless-padding-0-on-cell-44936.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus de 3 ans
aucun padding n'est déclaré en amont (il n'y a pas de reset à faire)
En testant sur une cellule JSON je constate bien cela, mais comment savoir que c'est bien toujours le cas ?
Mis à jour par Thomas Jund il y a plus de 3 ans
J'ai fais une recherche sur les sélecteurs suivants dans un fichier style.css compilé (core only):
div#services > ul > li, .a2-block, .cell, .block
Et aucun de ces sélecteurs n'a reçu de padding avant cette déclaration. La déclaration est donc à mon avis inutile.
Cela est valable uniquement si les CSS customs des thèmes surchargent les CSS du core (ce qui devrait être théoriquement le cas).
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Alors oui,
je constate comme toi que les fichiers CSS sont d'abord construits avec le contenu du core,
et qu'aucun des sélecteurs impliqués n'a reçu de padding avant cette déclaration.
Mis à jour par Thomas Jund il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ae6b57f8cbc787075edb9cae5eee55d6cc92dcdd Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Jul 3 17:16:42 2020 +0200 css: remove useless padding:0 on cell (#44936)
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
css: remove useless padding:0 on cell (#44936)