plugin.js: improve `ignore`

MohammadYounes / rtlcss

Framework for transforming Cascading Style Sheets (CSS) from Left-To-Right (LTR) to Right-To-Left (RTL)

https://rtlcss.com

MIT License

1.68k stars 129 forks source link

Closed XhmikosR closed 2 years ago

XhmikosR commented 2 years ago

use regex test since we only care about the boolean result and there's no global flag
remove temporary variable

The last 2 patches need more thorough review; I'm not 100% sure they are better, I'll let you decide. :)

MohammadYounes commented 2 years ago

I don't think these are equivalent. In original code, the endNode is being set before returning.

XhmikosR commented 2 years ago

I reverted the offending patch.