search

iamvdo / pleeease

Process CSS with ease: combine preprocessors and PostCSS
http://pleeease.iamvdo.me
474 stars 34 forks source link

css-mqpacker peut changer l'ordre des règles #17

Closed marvinroger closed 9 years ago

marvinroger commented 10 years ago

Salut ! (sympa de voir un français sur GitHub)

Je viens de signaler à l'auteur de css-mqpacker que ce package peut changer l'ordre des règles, et par conséquent casser le rendu de pages, le rendant inutilisable en production. (voir https://github.com/hail2u/node-css-mqpacker/issues/16)

En attendant que ce bug soit résolu, pleeease n'est donc par conséquent pas utilisable en production. Peut-être qu'il serait plus judicieux de supprimer cette fonctionnalité en attendant que cela soit résolu ?

iamvdo commented 10 years ago

Salut!

En attendant, tu peux surtout désactiver mqpacker dans Pleeease. Mais il est vrai que ce problème est sérieux, et compliqué à prendre à compte. Donc oui à terme il serait peut-être judicieux de le retirer, surtout que le gain n'est pas forcément si intéressant que cela, notamment car la compression GZIP marche très bien là-dessus.

MoOx commented 10 years ago

+1 about the limited utility vs the dangerosity of this tool. Doesn't bring any value to me.

oilvier commented 10 years ago

J'ai moi aussi rencontré un problème avec mqpacker que j'ai donc désactivé pour l'instant.

Du coup, j'aurais juste une question : est-ce que ça ne va pas géner le navigateur (d'une manière ou d'une autre) de croiser les mêmes media queries un peu partout dans la feuille de style plutôt que de n'avoir à les traiter qu'une seule fois (si elles sont regroupées via mqpacker) ?

MoOx commented 10 years ago

you should never ever change CSS author order or you will always have unexpected result that might happen. Cascade is broken.

oilvier commented 10 years ago

Yes, I get that. My intend was not to break the cascade, actually my goal with mqpacker was exactly to reproduce the cascade.

The real purpose of my post here is to know more about the interpretation of media queries by the browser.

MoOx commented 10 years ago

By literally moving some rules of their original place, you can't respect the original cascade.

Browsers don't have any issue to parse several mq everywhere.

iamvdo commented 10 years ago

+1. Pack MQs isn't safe. It's better to disable it. I think we need to set it to false for next version of Pleeease.

oilvier commented 10 years ago

Thanks for your feedback !

marvinroger commented 10 years ago

Great decision.