Produits Entr'ouvert » Combo

Trois colonnes = deux colonnes + barre latérale, ou autre chose ?

Si c'est bien ça, si c'est juste "bandeau" l'affaire, les placeholders arrivent de manière automatique, il suffit que le page_template_whatever.html de l'intégration graphique qui demande un bandeau contienne un placeholder bandeau.

L'idée est que le bandeau couvre les 2 colonnes.

Vu la capture, par "deux colonnes" tu entends "la largeur de la page les deux colonnes et la barre latérale aussi" ?

Le bandeau étant ici uniquement constitué d'une image, j'utiliserais l'illustration qui peut être attachée à la page, plutôt. (c'est pratiqué ainsi à Strasbourg).

Certes, en surchargeant le template et y rajoutant le placeholder on peut y arriver.

Je capte toujours pas vraiment ce ticket. Il s'agirait de faire :

+    'three-columns-and-banner': {
+        'name': _('Three columns and a banner'),
+        'template': 'combo/page_template_3cols_and_banner.html',
+    },

?

Frédéric Péters a écrit :

L'idée est que le bandeau couvre les 2 colonnes.

Faute de frappe: je voulais dire 3 colonnes.

Le bandeau étant ici uniquement constitué d'une image, j'utiliserais l'illustration qui peut être attachée à la page, plutôt. (c'est pratiqué ainsi à Strasbourg).

Oui. Mais on peut avoir des cas ou l'image ne prend pas toute la place. Une partie est dédiée à un bloc de texte (cf Amiens).

Donc au lieu de rajouter à chaque fois un template dans les variantes avec des placeholders personnalisés, avoir un template de base sur lequel on peut s'appuyer me parait une bonne chose.

Je capte toujours pas vraiment ce ticket.

Rajouter le template et des styles de base, le déclarer dans les settings,

Oui. Mais on peut avoir des cas ou l'image ne prend pas toute la place. Une partie est dédiée à un bloc de texte (cf Amiens).

Oui et donc sur une intégration pareille avoir un page_template_whatever.html de toute façon différent, adapté à l'intégration, plutôt que forcer un truc à tout le monde.

Rajouter le template et des styles de base, le déclarer dans les settings,

La réalité met ces templates et styles dans publik-base-theme.

Le patch ici, je ne vois toujours pas autre chose que mes quatre lignes. (et je n'en suis pas fan).

+    'homepage': {
+        'name': _('Homepage'),
+        'template': 'combo/page_template_home.html',
+    },

Et sans le template mais comme ça on peut dans un template simplement poser ce fichier pour que soit proposé un modèle différent (parce que généralement on parle de la page d'accueil les pages intérieures n'ont pas cette structure).

Frédéric Péters a écrit :

Oui et donc sur une intégration pareille avoir un page_template_whatever.html de toute façon différent, adapté à l'intégration, plutôt que forcer un truc à tout le monde.

Ce là que je voyais plutôt un travail à faire pour ne pas avoir des page_template_whatever.html dans les thèmes, quite à ne pas réproduire à 100% les maquettes.
Et que le travail se fasse principalement dans la feuille de style.

Frédéric Péters a écrit :

La réalité met ces templates et styles dans publik-base-theme.

Le patch ici, je ne vois toujours pas autre chose que mes quatre lignes. (et je n'en suis pas fan).

C'est ce que je viens de pousser dans la branche wip/template-3cols-topbanner
(https://git.entrouvert.org/combo.git/commit/?h=wip/template-3cols-topbanner&id=896fa5fe075bf03fd7157ff72fc72487164ce00e)
Le reste va dans publik-base-theme.

Ce là que je voyais plutôt un travail à faire pour ne pas avoir des page_template_whatever.html dans les thèmes, quite à ne pas réproduire à 100% les maquettes.

Mais c'est la réalité aujourd'hui, ça serait bizarre d'avoir page_template_3cols_and_banner.html dans combo alors que tous les autres sont dans publik-base-theme/templates/combo/, non ?

Et que le travail se fasse principalement dans la feuille de style.

Oui et c'est ce qui se passe.

Vraiment pas fan de la confusion qu'amènera l'affichage d'un placeholder "bandeau" en haut de la configuration des pages, sur toutes les instances, où ce n'est pas utilisé, qui arrivera en premier dans la configuration des pages.

+ 'topbanner-threee-columns': {

Même corrigé je continuerai vraiment à trouver ça pas terrible.

Je continue à m'expliquer.

Une intégration graphique demande un ajustement au format bandeau puis trois colonnes, Villeurbanne c'est "presque l'équivalent", Nancy l'apparence est pareille mais ce n'est pas du tout construit comme ça. Ce n'est pour moi pas si fréquent et il n'y a aucune difficulté à déclarer un modèle particulier sur un site donné.

Le seul bénéfice que je vois c'est ne pas avoir à déclarer ce modèle. Comme je ne suis pas convaincu que la structure précise est généralisée, ma proposition pour atteindre ce seul bénéfice, c'est de déclarer un modèle "Page d'accueil", qui me semble correspondre aux usages, et dont le nom marchera également pour l'accueil d'un site qui serait "bandeau & deux colonnes", genre.

Ici donc ça ajoute un format très particulier "trois colonnes + bandeau", alors qu'on n'a pas "trois colonnes", alors qu'on n'a pas "deux colonnes + bandeau", etc. Ça sonne totalement arbitraire. Si un modèle "trois colonnes", différent dans le sens du modèle "barre latérale + deux colonnes" est utile, pourquoi pas.

Sur une intégration graphique particulière, qui demanderait un bandeau, il est aussi très facile d'ajouter un placeholder "bandeau" dans les modèles existants (et quelqu'un travaillant sur le site aura une notion nette de la présence d'un bandeau, aura une idée nette de la zone de la page touchée par ce placeholder).

Par rapport aux captures, elles montrent plutôt pour moi un arrangement qui serait aujourd'hui réalisé avec grid 2/3 grid 1/3 pour la première ligne puis 3× grid 1/3 pour la deuxième (modulo que dans la capture la première ligne n'est pas fractionnée pareille, ne se cale pas sur une grille); un taf ici ça pourrait être de faciliter la disposition en grille (avec pourquoi pas des expérimentations avec css grid), un autre taf qui me vient à l'esprit sur ces captures, c'est la possibilité dans une zone d'avoir différentes sections (c'est un peu la demande #7445).

Toujours vraiment pas convaincu par ce ticket.

(traditionnellement on n'efface pas de données de redmine parce que ça peut rendre les commentaires passés qui y font référence compliqués à comprendre)

combo/settings.py

Ok. (même si dans combo/manager/ on utilise "home page", avec un espace).

combo/public/templates/combo/page_template_homepage.html | 1 +

Non, pour moi on veut permettre à une intégration graphique de proposer un modèle particulier pour la page d'accueil, on ne gagne rien à offrir un choix de modèle "page d'accueil" qui soit juste un alias.

Mauvais patch

(et sur le message de commit, rappel sur la forme, "add xxx" plutôt que "added xxx", et inclure une référence au ticket).

"settings: declare potential template for home page" comme intitulé ? (ack avec ça, ou autre variation)