Projet

Général

Profil

Development #74206

renommer pk-table-fit-contain → content (?)

Ajouté par Frédéric Péters il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
06 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Ça me donnait l'idée que c'était pour la faire corresponde au contenu mais la description dit "Donne au tableau la largeur du conteneur".

Donc plutôt pk-table-fit-container ?

Révisions associées

Révision 1d3b3d05 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ un an

css: use .pk-table-fit-container instead .pk-table-fit-contain (#74206)

Historique

#1

Mis à jour par Frédéric Péters il y a environ un an

  • Sujet changé de renommer pk-table-fit-contain → content à renommer pk-table-fit-contain → content (?)
#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ un an

Explication du choix

en css, la valeur `fit-content` désigne la largeur du contenu de élément. Ce qui est le cas sans cette class : la tableau a la largeur de son contenu.

Là on veut que le tableau prenne la largeur de conteneur parent

Je suis donc parti sur le css `object-fit: contain` pour donner `table-fit-contain`
Mais je comprends que ça ne veuille rien dire

Pourquoi pas `table-fit-parent` ?

#3

Mis à jour par Frédéric Péters il y a environ un an

Pourquoi pas `table-fit-parent` ?

La description dit "Donne au tableau la largeur du conteneur" je pense utile de limiter le nombre de termes employés, et que vu qu'on dit "conteneur" dans la description on devrait dire "container" dans le nom.

Cela étant dans https://doc-publik.entrouvert.com/admin-fonctionnel/modifier-le-contenu-des-portails/classes-css/ on ne parle ailleurs ni de "conteneur" ni de "parent", si on trouve que la description serait plus claire à parler de "parent" ou d'"élément parent", on peut tout à fait l'adapter.

#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ un an

Je préfère `container` que `parent` ou `parent-element`,
`container` se réfère à une forme (et comme on parle de largeur) alors que parent n'évoque rien de géométrique.
ok pour `pk-table-fit-container` ?

#5

Mis à jour par Frédéric Péters il y a environ un an

ok pour `pk-table-fit-container` ?

C'était ma suggestion dans la description du ticket donc ok de mon côté.

#6

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Thomas Jund (congés, retour le 29/04)

Thomas Jund (tjund) a ouvert une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a environ un an

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

Corentin Sechet (csechet) a approuvé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Thomas Jund (tjund) a mergé une pull request sur Gitea concernant cette demande :

#9

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#10

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF