Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

minify js et css #45

Open
B4rb3rouss opened this issue Jan 25, 2018 · 7 comments
Open

minify js et css #45

B4rb3rouss opened this issue Jan 25, 2018 · 7 comments

Comments

@B4rb3rouss
Copy link
Contributor

J'ouvre cette issue pour en discuter.
On avait parlé d'un système "automatique" qui minifierai les ressources css et js de blogotext.
J'ai commencé à travailler dessus un tout petit peu.

Je vois les choses ainsi pour l'instant, mais serait ravi d'avoir d'autres idées :

  • Appel d'un hook qui regarde le code de la page, et extrait dans le header les .css
  • Avec un fichier .php, on minifie et on met en cache
  • On remplace les appels dans le header vers les fichiers minifiés

Souci : le hook doit parser tout le html, avec les défauts et ralentissements que cela entraîne.
De plus, il faut mettre les fichiers css minifiés dans le même dossier que celui d'origine (si ces derniers ont des chemins relatifs dedans vers des fonts ou images)

@remrem
Copy link
Contributor

remrem commented Jan 25, 2018

Yop!
Déjà dans les tuyaux, ma faute, toujours pas pris le temps de pousser... :/

@B4rb3rouss
Copy link
Contributor Author

C'est vrai ? ♥♥♥
Trop super !
Bon, j'attends un peu alors ^^

@B4rb3rouss
Copy link
Contributor Author

Mine de rien, toutes tes améliorations, il y en a un paquet, ça va être une release épatante :)

@B4rb3rouss
Copy link
Contributor Author

Désolé pour le flood, juste une idée comme ça : selon la méthode que tu as mise en place pour minifier le css et le js, as-tu activé la compression gzip par php en même temps?

@remrem
Copy link
Contributor

remrem commented Jan 27, 2018

A quel niveau ?

@B4rb3rouss
Copy link
Contributor Author

C'est une question con sans doute.
Le fichier php qui génère le css ou le js minifié, il appelle bien "ob_start("ob_gzhandler");" ?

@remrem
Copy link
Contributor

remrem commented Jan 28, 2018

Yep. BT avait déjà ça en place, mais j'ai un peu modifier l'existant pour :

// check if client and server handle gzip
if (substr_count($_SERVER['HTTP_ACCEPT_ENCODING'], 'gzip') && extension_loaded('zlib')) {
    // set header
    header('Content-Encoding: gzip');
    // if cache not empty, clean it
    if (ob_get_length() > 0) {
        ob_end_clean();
    }
    // start
    ob_start('ob_gzhandler');
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants