search

jjlharrison / gulp-rtlcss

Gulp plugin that uses RTLCSS to convert LTR CSS to RTL.
MIT License
36 stars 7 forks source link

bug when "autoRename": false #2

Closed LyoNick closed 9 years ago

LyoNick commented 9 years ago

i.e try to set "autoRename": false inside a config file .rtlcssrc and run with this css:

.pull-left {left: auto;}
.pull-left {content: ' ';}

with the command-line rtlcss sample.css sample-rtl.css the result are:

.pull-left {right: auto;}
.pull-left {content: ' ';}

wrong result with gulp-rtlcss

.pull-left {right: auto;}
.pull-right {content: ' ';}

rtlcss version: 1.4.3

By default RTLCSS will auto rename any selector having directional keywords but with no directional declarations, gulp-rtlcss ignore the option to turn auto rename off completely. (perhaps it ignore the entire config file, i haven't tested it that far).

jjlharrison commented 9 years ago

Thank you. @MohammadYounes has kindly fixed this. I've also added a section to the README and released v0.1.4.