Projet

Général

Profil

Development #62305

forms_of_category cell: class .cell--body sur div.intro

Ajouté par Thomas Jund il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
01 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour uniformiser et éviter du code redondant.


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #62307: supprimer code specifique de div.introRejeté01 mars 2022

Actions

Historique

#1

Mis à jour par Thomas Jund il y a environ 2 ans

#2

Mis à jour par Thomas Jund il y a environ 2 ans

#3

Mis à jour par Frédéric Péters il y a environ 2 ans

Je ne sais pas si c'est approprié le <div> ici s'arrête pour contenir uniquement le texte, pas tout le contenu de la cellule.

#4

Mis à jour par Thomas Jund il y a environ 2 ans

C'est la sémantique de la class qui ne serait pas appropriée ? Ils ont les 2 les mêmes caractéristiques (même padding).
Une cell n'étant pas une personne, cela ne me dérange pas qu'elle puisse posséder plusieurs bodies, ou un body suivi d'un links-list puis à nouveau d'un body. Body étant là pour correctement positionner son contenu par rapport à l'enveloppe de la cellule.
(mais je comprends, néanmoins l'idée de supprimer du code redondant reste amha bonne).

#5

Mis à jour par Frédéric Péters il y a environ 2 ans

Oui perso cell--body j'imaginais qu'une cellule n'en aurait jamais qu'un. (et le contraire me dérange un peu).

(genre je trouverais pas délirant de me dire "ah tiens je veux le contenu de la cellule avec un fond bleu c'est la classe cell-body que je dois viser")

#6

Mis à jour par Thomas Jund il y a environ 2 ans

Oui perso cell--body j'imaginais qu'une cellule n'en aurait jamais qu'un

Ce serait coherent et bien, mais pour le moment, la structure des cellules ne le permet pas encore, car pour cela il faudrait pourvoir insérer une links-list au sein de cell--body, et ça demande encore un peu de boulot et de retouche des themes existants. C'est dans ces cells que j'imagine pouvoir avoir cell-body et links-list en cohabitation en attendant mieux (et parce que c'est ça factorise vers du mieux).

On peut aussi considérer que du code redondant c'est mieux et fermer ce ticket

#7

Mis à jour par Frédéric Péters il y a environ 2 ans

Je préfère la cohérence et ne pas introduire ce moment temporaire où cell--body ne serait pas tout le contenu; je préfère en effet qu'on garde les 5 lignes de #62307.

#8

Mis à jour par Thomas Jund il y a environ 2 ans

  • Statut changé de Solution proposée à Rejeté

Formats disponibles : Atom PDF