Projet

Général

Profil

Development #40223

Pouvoir définir une image dans les ressources pour la cellule "Liste de liens"

Ajouté par Stéphane Laget il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
27 février 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

0003-misc-reduce-querysets-on-page-detail-for-user.patch (985 octets) 0003-misc-reduce-querysets-on-page-detail-for-user.patch Lauréline Guérin, 10 mars 2020 16:40
0004-assets-display-assets-related-to-cell-in-popup-40223.patch (14,9 ko) 0004-assets-display-assets-related-to-cell-in-popup-40223.patch Lauréline Guérin, 10 mars 2020 16:40
0002-assets-compatibility-for-wcs-assets-40223.patch (8,72 ko) 0002-assets-compatibility-for-wcs-assets-40223.patch Lauréline Guérin, 10 mars 2020 16:40
0001-assets-add-generic-assets-for-cells-40223.patch (17,8 ko) 0001-assets-add-generic-assets-for-cells-40223.patch Lauréline Guérin, 10 mars 2020 16:40
0002-assets-compatibility-for-wcs-assets-40223.patch (8,72 ko) 0002-assets-compatibility-for-wcs-assets-40223.patch Lauréline Guérin, 12 mars 2020 11:53
0003-assets-display-assets-related-to-cell-in-popup-40223.patch (14,9 ko) 0003-assets-display-assets-related-to-cell-in-popup-40223.patch Lauréline Guérin, 12 mars 2020 11:53
0001-assets-add-generic-assets-for-cells-40223.patch (17,8 ko) 0001-assets-add-generic-assets-for-cells-40223.patch Lauréline Guérin, 12 mars 2020 11:53
0002-assets-compatibility-for-wcs-assets-40223.patch (8,72 ko) 0002-assets-compatibility-for-wcs-assets-40223.patch Lauréline Guérin, 16 mars 2020 11:12
0003-assets-display-assets-related-to-cell-in-popup-40223.patch (15,7 ko) 0003-assets-display-assets-related-to-cell-in-popup-40223.patch Lauréline Guérin, 16 mars 2020 11:12
0001-assets-add-generic-assets-for-cells-40223.patch (17,5 ko) 0001-assets-add-generic-assets-for-cells-40223.patch Lauréline Guérin, 16 mars 2020 11:12

Révisions associées

Révision 5d6d0a86 (diff)
Ajouté par Lauréline Guérin il y a environ 4 ans

assets: add generic assets for cells (#40223)

Révision 624120de (diff)
Ajouté par Lauréline Guérin il y a environ 4 ans

assets: compatibility for wcs assets (#40223)

Révision 90f17f85 (diff)
Ajouté par Lauréline Guérin il y a environ 4 ans

assets: display assets related to cell in popup (#40223)

Historique

#1

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.

#2

Mis à jour par Lauréline Guérin il y a environ 4 ans

  • Assigné à mis à Lauréline Guérin
#3

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

#4

Mis à jour par Lauréline Guérin il y a environ 4 ans

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 ?

#6

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.

#8

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.

#9

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 ?

#10

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

Yes, parfait, passé trop vite.

#11

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)
#12

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

Formats disponibles : Atom PDF