Project

General

Profile

Développement #44936

Développement #44934: Plan amélioration layout des templates

supprimer padding:0 inutile sur les cellules

Added by Thomas Jund over 4 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Thomas Jund
Target version:
-
Start date:
08 July 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.

Files

0001-css-remove-useless-padding-0-on-cell-44936.patch (689 Bytes) 0001-css-remove-useless-padding-0-on-cell-44936.patch Thomas Jund, 08 July 2020 11:38 AM

Associated revisions

Revision ae6b57f8 (diff)
Added by Thomas Jund over 4 years ago

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

History

#1

Updated by Thomas Jund over 4 years ago

#2

Updated by Nicolas Roche over 4 years ago

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

Updated by Thomas Jund over 4 years ago

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

Updated by Nicolas Roche over 4 years ago

  • Status changed from Solution proposée to 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

Updated by Thomas Jund over 4 years ago

  • Status changed from Solution validée to 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

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

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

Also available in: Atom PDF