Projet

Général

Profil

Development #32660

utiliser des URL absolues pour les /static/

Ajouté par Frédéric Péters il y a environ 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
28 avril 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans l'idée de pouvoir depuis combo adapter le style en générant de l'scss→css, l'URL servie ne sera pas sous /static/, il faut donc que des URL qui étaient aujourd'hui relatives (typiquement, les polices, ../fonts/...), soient cherchées de manière absolue.


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Bug #84816: Fichiers manquant quand on lint vaguement les fichiers style.cssNouveau17 décembre 2023

Actions

Révisions associées

Révision cb687167 (diff)
Ajouté par Frédéric Péters il y a environ 5 ans

general: use absolute path to fonts (#32660)

Révision cc1b4038 (diff)
Ajouté par Frédéric Péters il y a environ 5 ans

general: use absolute path to shared resources (#32660)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 5 ans

0001 pour les polices, 0002 pour quelques fichiers résiduels.

Hésitation restante sur l'utilisation de $static, plutôt que simplement écrire en dur /static/, l'avantage c'est un hypothétique fonctionnement mono-domaine où ces URL ne seraient plus en /static/; l'inconvénient c'est que ça a demandé de déplacer toute une série d'@import, et qu'il faudra aussi regarder aux intégrations d'atreal et d'imio.

Je ne touche pas aux URL d'images attachées à une intégration particulière, sur l'idée que le début du travail va se faire à partir des intégrations "neutres" (clapotis, grosboule, trifouilly).

#2

Mis à jour par Frédéric Péters il y a environ 5 ans

inconvénient c'est que ça a demandé de déplacer toute une série d'@import, et qu'il faudra aussi regarder aux intégrations d'atreal et d'imio.

Ça l'emporte, version qui met simplement en dur le /static/.

#3

Mis à jour par Nicolas Roche il y a environ 5 ans

(Désolé, je ne maîtrise pas le sujet.)
Je constate que tous les chemins relatifs ont été corrigés pour ces fichiers :

static/includes/_font-*.scss
static/includes/_forms.scss
static/includes/_utils.scss
static/includes/_wcs.scss
static/*/_custom.scss

Par contre plus loin dans l'arborescence, certains chemins restent inchangés :

$ find . -maxdepth 3 -type f -name '_*.scss' -exec grep -nH "\.\./" {} \; | grep -v '@import' | wc -l
0
$ find . -type f -name '_*.scss' -exec grep -nH "\.\./" {} \; | grep -v '@import' | wc -l
12

#4

Mis à jour par Thomas Noël il y a environ 5 ans

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

Ça me semble sans soucis.

#5

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit cc1b4038176a262f0d51945fdbd92966d15cecd7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Apr 28 09:21:02 2019 +0200

    general: use absolute path to shared resources (#32660)

commit cb687167edfba3c74ab75c3436d282f1df8f4bb7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Apr 30 09:32:49 2019 +0200

    general: use absolute path to fonts (#32660)
#6

Mis à jour par Frédéric Péters il y a presque 5 ans

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

Mis à jour par Benjamin Dauvergne il y a 5 mois

  • Lié à Bug #84816: Fichiers manquant quand on lint vaguement les fichiers style.css ajouté

Formats disponibles : Atom PDF