Produits Entr'ouvert » Intégrations graphiques Publik

J'avais cru comprendre que l'information "description" attachée aux pages (#23262) mais je ne vois pas ça dans le patch, il lui manque un fichier ? (qui gouvernerait la cellule de type menu prévue dans le placeholder bottom, si je lis bien la css).

Pourquoi ne pas aussi en avoir profité pour le carrousel (d'ailleurs, attention à l'orthographe contre-intuitive, même pas l'objet de réformes récentes) de haut de page ? S'éviter ainsi l'ajax, voire tout javascript. La structure aurait ainsi pu être <a href="page.get_online_url" style="background-image: url({{ page.image.url }} )">{{page.description}}</a> (voire même la description utilisée en alt, en supposant le texte comme faisant partie de l'image). overflow hidden, dimensions fixées pour le <a>, scroll-behaviour: smooth, ça aurait pu être pas mal, chic et pas cher. (une variante carrousel officielle pour combo fera(it) ça).

Aussi sur ce haut de page, plutôt qu'y demander l'ajout d'une cellule de type menu, le code aurait totalement pu être incorporé au template, je trouve.

_~~

(des erreurs d'indentation)

@include flexbox(); puis flex-flow: row wrap;

L'idée des mixins flexbox devait être de permettre la compatibilité MSIE sans se poser trop de questions, ce bénéfice est perdu si on en vient à mentionner des propriétés flex de manière indépendante.

Pour les sélecteurs de pseudo-élément, type ::after, la spec dit deux fois deux points.

(passons)

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

J'avais cru comprendre que l'information "description" attachée aux pages (#23262) mais je ne vois pas ça dans le patch, il lui manque un fichier ? (qui gouvernerait la cellule de type menu prévue dans le placeholder bottom, si je lis bien la css).

Template zappé...

Pourquoi ne pas aussi en avoir profité pour le carrousel (d'ailleurs, attention à l'orthographe contre-intuitive, même pas l'objet de réformes récentes) de haut de page ? S'éviter ainsi l'ajax, voire tout javascript. La structure aurait ainsi pu être <a href="page.get_online_url" style="background-image: url({{ page.image.url }} )">{{page.description}}</a> (voire même la description utilisée en alt, en supposant le texte comme faisant partie de l'image). overflow hidden, dimensions fixées pour le <a>, scroll-behaviour: smooth, ça aurait pu être pas mal, chic et pas cher. (une variante carrousel officielle pour combo fera(it) ça).

Je me suis peut-être trop pris la tête, mais la position du texte dans les images du carroussel pourrait varier en fonction de chaque image et je préfèrais laisser cette tâche à ceux qui éditeront le contenu.

Ce que tu as fait dans #23520 est la bonne piste à prendre. Je préfère prendre cela en compte dans un deuxième temps.

Aussi sur ce haut de page, plutôt qu'y demander l'ajout d'une cellule de type menu, le code aurait totalement pu être incorporé au template, je trouve.

Mon idée était de faciliter l'édition du contenu: au lieux des liens posés dans le template, pourvoir modifier l'intitulé, le contenu facilement dans le backoffice.

(des erreurs d'indentation)

corrigées.

@include flexbox(); puis flex-flow: row wrap;

L'idée des mixins flexbox devait être de permettre la compatibilité MSIE sans se poser trop de questions, ce bénéfice est perdu si on en vient à mentionner des propriétés flex de manière indépendante.

J'ai rajouté la déclaration de flex-wrap dans #23522.

Pour les sélecteurs de pseudo-élément, type ::after, la spec dit deux fois deux points.

aussi.

Caroussel → Carrousel.

Pour les templates j'aurais imaginé ne pas se soucier des sous-menus (parce qu'en pratique, aucune idée du rendu).

$font-family: Din, Roboto;

fallback sur sans-serif quand même ?