Project

General

Profile

Evolution #3425

Appliquer compresseur_embarquer_images_css à l'aide de la globale compresseur_filtres_css

Added by b b over 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
04/14/2015
Due date:
% Done:

0%

Resolution:
fixed

Description

Ce matin je découvrais cette fonctionnalité cachée du compresseur, cf :

http://zone.spip.org/trac/spip-zone/changeset/37349/

Pour info, en l'état elle ne peut pas fonctionner malgré la déclaration qui va bien dans mes_options par ex :

$GLOBALS['compresseur_filtres_css'] = array('compresseur_embarquer_images_css');

En effet, inc/compresseur_embarquer n'est jamais chargé depuis inc/compresseur, du coup il ne se passe rien.

J'ai testé en ajoutant l'include_spip qui va bien ici :

http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/compresseur/inc/compresseur.php#L188

Avec cette modif, le filtre est bien appliqué. Du coup je me dis qu'il serait intéressant de commiter ça. Des avis sur la question ?

Pour continuer, d'après mes essais sur une 3.0.x up du jour, le filtre en question n'effectue aucun changement sur des règles simples telles que :

.article_fiche .menu.contact .item.adresse { background-image: url(images/adresse.png); }

J'ai bien lu que la fonctionnalité en question est expérimental, mais ça fait maintenant 5 ans, du coup je me dis qu'on pourrait s'y pencher :)

History

#1 Updated by cedric - over 4 years ago

  • Target version changed from 3.0 to 3.1

#2 Updated by b b over 4 years ago

  • Tracker changed from Anomalie to Evolution

#3 Updated by cam.lafit - over 4 years ago

Ciao

Dans ce cas il serait bon d'ajouter :
  • l'include dans la branche 3.0.x pour permettre aux gens
    d'expérimenter comme il faut.
  • l'include et la globale pour la 3.1 ça serait une évolution de plus
    par défaut, autant profiter du changement de version pour activer pour
    de vrai ce truc

#4 Updated by b b over 4 years ago

Comme je le disais, le filtre n'est pas fonctionnel, du coup je pense qu'il faut limiter cette évolution à la branche 3.1 pour l'instant (donc ajouter l'include manquant uniquement en 3.1). On aura tout le temps de reporter en 3.0 le jour où ça fonctionnera et qu'on aura bien testé le bouzin ;)

#5 Updated by cam.lafit - over 4 years ago

Ciao

Si j'ai bien compris compresseur_embarquer ne peut fonctionner ni en
3.0, ni en 3.1 parcequ'il manque l'inclusion. Et c'est un truc noté
expérimental depuis 5 ans. Si j'ai bien compris alors :
  • il me semble logique de corriger l'oubli en 3.0 pour permettre aux
    gens de tester ce mode, donc de rendre juste vrai le commentaire du
    commit
  • d'activer ce mode en 3.1 pour réellement le tester, autrement cela
    restera encore caché pour les 5 ans à venir et alors autant le retirer
    du core n'apportant rien.

Si je n'ai rien compris, ne pas lire ce message :)

#6 Updated by cedric - almost 4 years ago

Difficile d'activer cette fonction car l'embarquement des images dans les CSS est incompatible avec les IE<=7 http://www.websiteoptimization.com/speed/tweak/inline-images/

#7 Updated by cedric - almost 4 years ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

#8 Updated by b b almost 4 years ago

\o/ merci

Also available in: Atom PDF