Produits Entr'ouvert » Intégrations graphiques Publik

Garbage de bout de fichier.

Cacher le <h2> me semble hors de propos pour ces classes. (je peux bien sûr comprendre l'intérêt de le cache, mais via une classe différente)

Quand il y a des bordures arrondiers, ça s'empile assez mal (essayer avec l'ancien thème Fontenay par exemple).

Et concernant les noms des classes, je me dis qu'on peut mieux faire, genre no-bottom-margin me semble plus facile que bottom-glued.

Avoir deux classes ça me pose déjà question et j'imagine que ça en posera encore plus tard; idéalement pour moi on aurait uniquement no-margin-bottom, sans nécessité d'expliquer qu'il faut à la fois no-margin-bottom et no-margin-top, ou sans nécessité d'expliquer qu'il n'est pas nécessaire d'avoir les deux.

Ça montre aussi que les mises à zéro des marges des <p> posent problème, si je comprends l'intention, la réduction devrait uniquement concerner la marge de bas du dernier <p>.

Il me semble que ça reste d'application. On aurait par exemple une cellule de type texte, avec dedans plusieurs paragraphes, on n'a pas envie que les marges de tout ceux-ci soient altérées.

+                       p:first-child {
+                               margin-top: 0;
+                       }

Là où ça vire le margin-top, et pareil pour le margin-bottom, le remplacer par un padding-top (resp. padding-bottom) pour que les textes ne se trouvent pas collés.

(et j'avais cru le noter mais il y a des espaces mal placés dans ce patch, après une première tabulation, git am le signale et git show les affiche).

(sur mon exemple, appliquer le padding-top sur le p:first-child fonctionne; je n'ai pas testé toutes les situations possibles)

Dans ma situation, cellule avec no-bottom-margin puis cellule texte avec deux paragraphes, le premier se trouve pris :first-child et donc margin-top: 0 et donc il est collé à la séparation horizontale de la cellule précédente.

Et donc à nouveau, sur mon exemple, appliquer le padding-top sur le p:first-child fonctionne; je n'ai pas testé toutes les situations possibles.

Le contre-exemple que tu donnes, c'est l'enchainement de deux cellules textes, les margin-bottom/top y étant remplacé par des padding-bottom/top, ceux-ci s'additionnent et il y a donc le double d'espace. Je serais à dire que cette situation d'ajout de deux cellules textes l'une sous l'autre est moins importante (après tout, il suffit de taper le tout dans une seule cellule) (le cas qui justifierait deux cellules serait des permissions différentes).