Projet

Général

Profil

Development #47594

Gestion des ressources dans le thème de Sens

Ajouté par Nicolas Roche il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour pouvoir changer l'image (et le logo).


Fichiers

Révisions associées

Révision f358e700 (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

sens: use assets for banner and logo (#47594)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 3 ans

#3

Mis à jour par Frédéric Péters il y a plus de 3 ans

    background: var(--page-picture, url(/assets/header:background));

Au cas où, ça ne marche pas avec IE de faire ça; je laisserais juste url(/assets/header:background); si tu veux vraiment que le --page-picture fonctionne, tu dois overrider page_template.html et y définir le bloc msie_css_page_picture adéquat, cf :

{% block msie_css_page_picture %}
{# IE11 doesn't support CSS variable and supports() API, so we set it here #}
{# and themes that do need a different CSS will have to redefine the       #}
{# msie_css_page_picture block.                                            #}

S'il n'y a pas de demande explicite de faire varier l'image de page en page, je zapperais juste tout ça.

#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

Ou alors tu assumes que l'image ne s'affiche pas sous IE (si elle n'a qu'un rôle de décoration est n'apporte ni fonctionnalité ni information).
Dans ce cas, il faut poser également une background-color pour IE.
Ou encore tu doubles l'information et prévoit que header:background soit pour IE

background-image: url(/assets/header:background);
background-image: var(--page-picture, url(/assets/header:background));

IE ne comprenant la seconde ligne va se contenter de la première.

Mais dans l'état, comme le dit Fred, là tu n'as rien sous IE.

#5

Mis à jour par Nicolas Roche il y a plus de 3 ans

ça ne marche pas avec IE de faire ça

merci pour le warning

S'il n'y a pas de demande explicite de faire varier l'image de page en page, je zapperais juste tout ça.

oui ça me va, comme ça je retombe sur l'utilisation standard des assets.

#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#7

Mis à jour par Nicolas Roche il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit f358e7002624f2128f040688b55c69c45f5e58c1
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sat Oct 10 02:21:21 2020 +0200

    sens: use assets for banner and logo (#47594)
#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF