songsiqi / px2rem

According to one stylesheet, generate rem version and @1x, @2x and @3x stylesheet.
558 stars 92 forks source link

Support `/*!no*/` comments syntax to fix issues #2 #9

Open leiming opened 8 years ago

leiming commented 8 years ago

提供 /*!no*/ 的注释语法,用于支持less-loader

请容我解释一下为何在px2rem内部兼容,而不是去修复 webpack 或者 less-loader:

必须通过 ! 兼容 Less.js 的 source-map ,而在 webpack loader 体系不能传递 !,这就陷入一个死循环。

我承认这个 PR 不是最优雅的做法,但在不修改 YUI Compress 标准,又不变化 Webpack 1.x 体系的前提下,在内部支持可能是代价最小却能解决实际问题的做法。

@songsiqi @Jinjiang 怎么看 ^_^?

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling c5c269eb248b188ff708e112f9a3b0767dd5b6a7 on leiming:fix-issues-2 into 20e30faa399c1797357968b6c2c62f134ebe184b on songsiqi:master.

leiming commented 8 years ago

Fix https://github.com/songsiqi/px2rem/issues/2 https://github.com/vuejs/vue-loader/issues/227

xiaoai7904 commented 5 years ago

可以试试把px更换为PX,我在react 项目中使用了该方式是没有问题的,希望对你有所帮助