Projet

Général

Profil

Development #44936

Development #44934: Plan amélioration layout des templates

supprimer padding:0 inutile sur les cellules

Ajouté par Thomas Jund 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:
08 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision ae6b57f8 (diff)
Ajouté par Thomas Jund il y a plus de 3 ans

css: remove useless padding:0 on cell (#44936)

Historique

#1

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

#2

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 ?

#3

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

#4

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.

#5

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)
#6

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