Development #62415
paramètre pour qu'un placeholder ajoute un <div>
0%
Description
Pou réaliser #62413 qui ajouterait des zones columns-top & colums-bottom, la forme actuelle serait de l'ordre de
<div class="columns-bottom">{% placeholder "columns-bottom" name=_('Bottom of content') optional=True acquired=False %}</div>
Mais ça va produire un <div> vide quand pas utilisé. On s'en passerait bien. (ça rassure aussi sur les éventuelles régressions sur des sites qui verraient soudainement un nouveau <div>, ajouteraient de la marge, que sais-je).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Development #62413: ajouter des zones "haut du contenu" et "bas du contenu" ajouté
Mis à jour par Frédéric Péters il y a environ 2 ans
(à noter que ça vient après #62072, dont l'option ajoute déjà un div).
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-misc-add-option-for-placeholders-to-create-an-outer-.patch 0001-misc-add-option-for-placeholders-to-create-an-outer-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund il y a environ 2 ans
J'aime beaucoup cette idée que les placeholders aient des balises dédiés.
L'id est inutile si class unique.
Juste pour être sûr: le template du placeholder ne sera généré que si une cellule existe ? (si oui c'est top).
Mis à jour par Thomas Jund il y a environ 2 ans
Et je serais pour dissocier le tag du placeholder sur lequel on peut associer des styles, au tag .fx-grid, qui gère les gouttières via margin negatifs.
Comme proposé ici https://dev.entrouvert.org/issues/62142#note-1 (2e option proposée)
Parce qu'il va arriver de vouloir associer des margins aux `.combo-placeholder` et ça va entrer en conflit.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-misc-add-option-for-placeholders-to-create-an-outer-.patch 0001-misc-add-option-for-placeholders-to-create-an-outer-.patch ajouté
Voilà le patch revu pour que la classe fx-grid reste dans un <div> spécifique à l'intérieur.
Ça pourrait donc donner :
<footer class="combo-placeholder combo-placeholder--footer"> <div class="fx-grid"> [des cellules] </div> </footer>
Mis à jour par Thomas Jund il y a environ 2 ans
Merci, cela garanti qu'aucun style ne soit ajouté à ".fx-grid".
(Je laisse un autre dev valider la partie pyhton)
Mis à jour par Serghei Mihai il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b02b7b0e5b79ca5ca0af61563157f5faf340f197 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Mar 4 08:18:31 2022 +0100 misc: add option for placeholders to create an outer tag (#62415)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add option for placeholders to create an outer tag (#62415)