search

SlexAxton / css-colorguard

Keep a watchful eye on your css colors.
2.44k stars 65 forks source link

update color name replacement regex #21

Closed patrickkettner closed 10 years ago

patrickkettner commented 10 years ago

currently in master, given the following css

#crazy{color:black;background-color:purple}

black and purple are matched as :black; and purple}, which means it gets transformed into

#crazy { color#000000background-color#800080

which causes rework to vomit

this pours pepto all over rework's tum tum

SlexAxton commented 10 years ago

Did you break the tests or did that already happen?

patrickkettner commented 10 years ago

I broke it, checking why now

patrickkettner commented 10 years ago

so that was a pain...

Easiest way to show what happened is with a bunch of links

original last pr now

patrickkettner commented 10 years ago