Produits Entr'ouvert » Combo

Je dois rater quelque chose : en quoi une cellule texte "info pratique" ne suffirait pas ?

Pour des raisons de style / ordre des éléments, ce texte doit s'insérer à l'intérieur d'une cellule.

Ce que je n'aime pas dans ce qui est présenté ici, c'est qu'il faut poser un placeholder dans le template de la page. Ca serait super que la cellule indique à la page qu'elle peut contenir un ou des placeholder, et que quand on ajoute ce genre de cellule, hop, automatiquement on voit apparaitre les placeholders (même si j'avoue j'ai rien regardé pour savoir si c'est jouable). Car oui, ça résoudrait un soucis important (avoir du texte "en dur" dans certaines cellules, non éditable).

Je n'avais pas du tout pensé à ça et j'étais parti pour écrire que c'était sans doute jouable mais du coup, à vérifier ça, je l'ai fait, et du coup le patch devient "découvre aussi les placeholders au sein des cellules". Il y a quand même quelques complexités, sur un exemple de cellule factures, par exemple, on aurait quelque chose de ce type :

  {% if invoices %}
    <p>Voici la liste de vos factures.</p>
  {% endif %}

qu'on voudrait transformer en :

  {% if invoices %}
    {% placeholder "intro-factures" name="Texte au-dessus des factures" %}
  {% endif %}

mais lors de la phase de découverte dans le manager on n'a pas envie de récupérer les factures de l'admin, et peut-être qu'il n'en n'a pas.

Du coup, le modèle va plutôt devoir être :

  {% placeholder "intro-factures" name="Texte au-dessus des factures" render=False %}
  {% if invoices %}
    {% placeholder "intro-factures" %}
  {% endif %}

C'est-à-dire commencer par déclarer qu'il va peut-être contenir un tel placeholder.

Le résultat me semble quand même tout à fait acceptable (ce n'est pas comme dans le template, tout à fait déconnecté).

Une limitation au patch que je présente, mais je préférerais vivre avec pour le moment, c'est que les noms de placeholder doivent être uniques, et on ne peut dès lors pas avoir sur la même page deux cellules du même type déclarant le même placeholder. (en fait on peut mais les cellules partageront le même contenu de placeholder). Sur les situations rencontrées, là où le bénéfice d'une zone éditable se faisait sentir, je ne pense pas que ça soit une limitation importante.

Chicos. Je pense qu'il manque des tests non ? J'ai essayé "en vrai", ça semble ok. Bien sûr il ne faut pas oublier le {% load combo %} quand on veut ajouter des placeholder, c'est normal. Le rendu HTML va donner des cellules imbriquées (et c'est bien logique, elles le sont) : sera-ce facile à gérer selon toi ?

Yep, un peu de tests demain. Sur le rendu il y aura sans doute une base de style "cellule dans une cellule" à mettre, je pense notamment qu'il faudra éliminer un peu de marges.

Ack

Peut-être que ça pourrait être le moment de mettre à jour le README sur ces histoires de placeholder ? (mais dans un ticket à venir, ça me va aussi, le README n'est pas faux ici)