Development #41995
inclure classe css pour indiquer qu'une cellule a des ressources associées
0%
Description
Dans #41741#note-9 il est exprimé qu'il serait utile pour le rendu d'une cellule d'avoir une classe CSS posée dessus indiquant que la cellule contient une image.
Je me dis qu'on peut get_asset_slots() et regarder s'ils sont tous définis pour la cellule et si c'est le cas, marquer la cellule et ajouter une classe à son div.cell.
Côté coût une fois qu'on a l'ensemble des cellules pour la page, on peut faire un get_asset_slots() dessus, puis en une seule requête sur assets.Asset obtenir ceux qui sont définis, ça augmenterait le rendu d'une page d'une seule requête SQL, il me semble.
(on pourrait aussi avoir une variante de get_asset_slots qui retourne juste les clés, histoire d'éviter tout le code de substitution de chaine utilisé pour les labels mais je ne sais pas si ça vaut la peine)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #41741: valeur pour $cell-image-position pour positionner l'image à gauche du titre ajouté
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-assets-add-css-classes-related-to-assets-on-cells-41.patch 0001-assets-add-css-classes-related-to-assets-on-cells-41.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fbda225b1aaafa816939403d3114df0ebaecbe32
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Apr 24 14:12:28 2020 +0200
assets: add css classes related to assets on cells (#41995)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans
- Lié à Development #42842: Appliquer styles de position de l'image d'une cell seulement s'il y a une image ajouté
assets: add css classes related to assets on cells (#41995)