Development #74206
renommer pk-table-fit-contain → content (?)
0%
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
Historique
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 (?)
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` ?
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.
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` ?
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é.
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 :
- URL : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/161
- Titre : css: use .pk-table-fit-container instead .pk-table-fit-contain (#74206)
- Modifications : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/161/files
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 :
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 :
- URL : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/161
- Titre : css: use .pk-table-fit-container instead .pk-table-fit-contain (#74206)
- Modifications : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/161/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
css: use .pk-table-fit-container instead .pk-table-fit-contain (#74206)