Development #51799
calvados, portail usager : rendre le logo modifiable par un admin fonctionnel
0%
Description
poser un asset comme logo
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 3 ans
- Fichier 0001-calvados-cd14-use-asset-for-header-and-emails-logo-5.patch 0001-calvados-cd14-use-asset-for-header-and-emails-logo-5.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
.... et faire de même pour les mails, et donc supprimer complétement le png devenu inutile
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
(Pas testé mais) si asset header:logo utilisé
`header:logo` est déjà injecté dans le background du lien par _header.scss
Théoriquement, t'as juste à modifier sa position :
background-position: bottom left;
Mis à jour par Thomas Noël il y a environ 3 ans
Thomas Jund a écrit :
(Pas testé mais) si asset header:logo utilisé
`header:logo` est déjà injecté dans le background du lien par _header.scss
Je n'ai pas compris... là je change juste le lien vers l'image en utilisant plutôt celle de l'asset. Ca suffit pas ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
Je n'ai pas compris.
Je voulais dire que si tu utilises l'asset, le core CSS l'injecte déjà, tu n'as pas besoin de le réinjecter en custom.scss
Ca suffit pas ?
heu si, ça marche. Je veux signifier que c'est même un peu trop du coup. :)
Et tu peux aussi rm img/logo-calvados-transparent.png
Mis à jour par Thomas Noël il y a environ 3 ans
Thomas Jund a écrit :
Je veux signifier que c'est même un peu trop du coup. :)
Je préfère être conservateur... Bon, en vrai je ne vois pas ce qu'il faut faire et je n'ose pas le dire ;)
Et tu peux aussi rm img/logo-calvados-transparent.png
Yep, c'est dans le patch, mais c'est discret : « delete mode 100644 static/calvados-cd14/img/logo-calvados-transparent.png »
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Fichier 0001-calvados-cd14-use-asset-for-header-and-emails-logo-5.patch 0001-calvados-cd14-use-asset-for-header-and-emails-logo-5.patch ajouté
Si on peut éviter de surcharger inutilement le core je préfère, j'y gagne en maintenance :)
Je te propose ce patch.
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Okaaayyyyyy j'avais vraiment pas compris ça, nickel, merci.
commit c46010d7cb1bbeae66308070d957469fed5bf6ef
Author: Thomas NOËL <tnoel@entrouvert.com>
Date: Mon Mar 8 16:14:00 2021 +0100
calvados-cd14: use asset for header and emails logo (#51799)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
calvados-cd14: use asset for header and emails logo (#51799)