px2rem icon indicating copy to clipboard operation
px2rem copied to clipboard

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

Open leiming opened this issue 8 years ago • 3 comments

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

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

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

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

@songsiqi @Jinjiang 怎么看 ^_^?

leiming avatar Sep 24 '16 22:09 leiming

Coverage Status

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

coveralls avatar Sep 24 '16 22:09 coveralls

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

leiming avatar Sep 24 '16 22:09 leiming

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

xiaoai7904 avatar Dec 29 '18 16:12 xiaoai7904