support to ignore comments eg. /*no*/ /*px*/

[lcat]

Sometimes selectorBlackList is not easy to use.

example:

// before - add selectorBlackList ['ignore']
.box {
    width: 10px;
    height: 10px;
}
.box.ignore {
    border: solid 1px #ddd;
}

// now - with comments
.box {
    width: 10px;
    height: 10px;
    border: solid 1px #ddd; /*no*/
}

[KODerFunk]

@lcat hi! no and px is too common words. Also need test.

[lcat]

@KODerFunk hei~ Test cases have been added, no and px is a reference https://github.com/songsiqi/px2rem

[KODerFunk]

@lcat hei! Tests are good! But I still worry about the correct expected result of the plugin. First, no and px are too simple common words. Secondly, we just need to be able to work without conflicts with other postcss-plugins.

Prease, 1) replace no and px to postcss-px-to-viewport-ignore 2) describe postcss-px-to-viewport-ignore comment directive in example in README.md and README_CN.md

[likun7981]

@lcat Hi guys, when are you planning to have this merged?

[KODerFunk]

@likun7981 This is a good feature! I’ll try to find the time this month to complete and release the update, but I need a translation for the readme before release, I will call you there later.

[hsxfjames]

@KODerFunk Hi, how is it going now?

And, how about the comments like px-to-viewport-disable or px-to-viewport-ignore? Shorter without postcss- prefix.

[KODerFunk]

@hsxfjames good idea with naming. I working on this feature by correct post-css api. The next release of these days.

[KODerFunk]

https://github.com/evrone/postcss-px-to-viewport/commit/9d6c5cee5eea367fb4c7cf0c3bb1117979b5fbf4

• Added /* px-to-viewport-ignore */ and /* px-to-viewport-ignore-next */ — special comments for ignore conversion of single lines.

Read more about ignoring.

需要翻译帮助。