Development #40223
Pouvoir définir une image dans les ressources pour la cellule "Liste de liens"
0%
Description
Pouvoir définir dans les ressources une image attachée à la cellule "Liste de liens" pour qu'elle puisse se présenter de la même façon qu'une cellule "démarches d'une catégorie", sur les thèmes où cette image est possible.
Ceci serait vraiment utile pour organiser le portail citoyen.
Fichiers
Révisions associées
assets: compatibility for wcs assets (#40223)
assets: display assets related to cell in popup (#40223)
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
On a déjà WCS_CATEGORY_ASSET_SLOTS et WCS_FORM_ASSET_SLOTS, je me dis qu'on peut arrêter là et faire un générique COMBO_CELL_ASSET_SLOTS,
COMBO_CELL_ASSET_SLOTS = { "LinkListCell": { "picture": { "prefix": "Image", }, "picture-hover": { "prefix": "Image", "suffix": "(survol)" } } }
De là, un get_asset_slots générique qui assemble ça avec des noms de slot comme "cell:<type de cellule>:<type de ressource>:<slug de la cellule>" (→ ex: cell:LinkListCell:picture-hover:pied-de-page); ça impose un slug aux cellules si on veut leur attribuer une image, ça me semble une exigence raisonnable.
On pourrait aussi se dire qu'on ne prend pas nécessairement le slug, qu'on aurait plutôt une méthode get_slug_for_asset(), qui pourrait être surchargée dans les cellules où un identifiant "unique/stable" existe déjà, comme le slug de la démarche liée.
Et se dire aussi que cell:<type de cellule> pourrait également être surchargé, et ainsi le code aujourd'hui spécifique dans WcsCommonCategoryCell et WcsFormCell pourrait pour ainsi dire disparaitre, tout en gardant des assets nommés pareil. (mais on peut se dire que la reprise de cette partie-là est pour un ticket ultérieur).
Comme libellé pour l'asset, prefix + "représentation textuelle de la cellule" + suffix; pour la représentation textuelle, je dirais que "type de cellule sur la page ... (slug)" (ex: "liste de liens sur la page Accueil (pied-de-page) (survol)").
~~
Parallèlement je me dis qu'on pourrait gagner une popup dans les liens sous une cellule, qui listerait les ressources associées, et permettrait de taper une image sans avoir à passer par la liste globale des ressources.
Mis à jour par Stéphane Laget il y a environ 4 ans
Frédéric Péters a écrit :
Parallèlement je me dis qu'on pourrait gagner une popup dans les liens sous une cellule, qui listerait les ressources associées, et permettrait de taper une image sans avoir à passer par la liste globale des ressources.
Et cela serait nettement plus compréhensible pour les utilisateurs
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0004-assets-display-assets-related-to-cell-in-popup-40223.patch 0004-assets-display-assets-related-to-cell-in-popup-40223.patch ajouté
- Fichier 0003-misc-reduce-querysets-on-page-detail-for-user.patch 0003-misc-reduce-querysets-on-page-detail-for-user.patch ajouté
- Fichier 0002-assets-compatibility-for-wcs-assets-40223.patch 0002-assets-compatibility-for-wcs-assets-40223.patch ajouté
- Fichier 0001-assets-add-generic-assets-for-cells-40223.patch 0001-assets-add-generic-assets-for-cells-40223.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Bon ben j'étais lancée, j'ai aussi fait la gestion du legacy pour les cellules w.c.s. et la popup des assets liés à une cellule
Je retire le patch 3 pour en fait un ticket à part ?
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0003-assets-display-assets-related-to-cell-in-popup-40223.patch 0003-assets-display-assets-related-to-cell-in-popup-40223.patch ajouté
- Fichier 0002-assets-compatibility-for-wcs-assets-40223.patch 0002-assets-compatibility-for-wcs-assets-40223.patch ajouté
- Fichier 0001-assets-add-generic-assets-for-cells-40223.patch 0001-assets-add-generic-assets-for-cells-40223.patch ajouté
avec le commit de perfs en moins
Mis à jour par Frédéric Péters il y a environ 4 ans
Je retirerais la possibilité par défaut de data_textcell, data_jsoncell et data_linkcell. Dans la popup j'aurais dit de ne pas reprendre le nom complet de la ressource, juste préfixe/suffixe, mais ça me semble plutôt galère et perdre le bénéfice d'un manager_assets_fragment.html clair et unique, j'oublie.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0003-assets-display-assets-related-to-cell-in-popup-40223.patch 0003-assets-display-assets-related-to-cell-in-popup-40223.patch ajouté
- Fichier 0002-assets-compatibility-for-wcs-assets-40223.patch 0002-assets-compatibility-for-wcs-assets-40223.patch ajouté
- Fichier 0001-assets-add-generic-assets-for-cells-40223.patch 0001-assets-add-generic-assets-for-cells-40223.patch ajouté
remarques prises en compte
(settings.py nettoyé, short_label ajouté pour le cas assets d'une cellule)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Me vient un truc, que je note pour archive; les dictionnaires imbriqués ainsi ne vont pas pouvoir facilement être altérés via un config.json d'une intégration graphique, exemple "WCS_FORM_ASSET_SLOTS.update": {"logo": {"prefix": "Pictogramme"}}, le .update permet d'ajouter la ressource "logo", mais ici on pourra juste faire COMBO_CELL_ASSET_SLOTS.update et ça demandera de redéfinir totalement les ressources associées à un type de cellule.
Mais aujourd'hui ça signifie au pire devoir mentionner l'asset "picture", ce n'est pas bien important de bloquer là-dessus. (et on pourrait se dire que la syntaxe .update de hobo pourrait évoluer pour permettre un COMBO_CELL_ASSET_SLOTS.update['wcs_wcsformcell'] ou autre truc du genre.
~~
+ short_label = u'%(prefix)s%(suffix)s' % {
Plutôt '%(prefix)s (%(suffix)s)' (en préfixe on aura "Image" et en suffixe quelque chose comme "survol").
Ok avec ce changement.
Mis à jour par Lauréline Guérin il y a environ 4 ans
Le suffix ici vaut:
suffix = '' if slot_template_data.get('suffix'): suffix = ' (%s)' % slot_template_data['suffix']
Donc
'%(prefix)s%(suffix)s'
c'est bon non ?Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 90f17f8595602ec834e966c61e5cdc731ae6d975 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Mar 10 16:39:04 2020 +0100 assets: display assets related to cell in popup (#40223) commit 624120dedfc09f0d8c4790bd2968675a5451b870 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Mar 10 14:12:49 2020 +0100 assets: compatibility for wcs assets (#40223) commit 5d6d0a86fc5377da135830d3bca3181e43904575 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Mar 10 09:52:41 2020 +0100 assets: add generic assets for cells (#40223)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
assets: add generic assets for cells (#40223)