Fixed #41 (Mark.pipes.sanitize) by adding or's to the regex; added tests

adammark / Markup.js

Powerful JavaScript templates

317 stars 51 forks source link

Fixed #41 (Mark.pipes.sanitize) by adding or's to the regex; added tests #42

Closed mdix closed 7 years ago

mdix commented 9 years ago

Fixed and tested, please merge. If you have a better solution for the input <-> output mapping than the empty elements I used please let me know.

adammark commented 9 years ago

Thanks—I'll review this ASAP.

mdix commented 9 years ago

I needed another feature, so I added it: A filter that replaces a given string with another string. I called it str_replace. Maybe you can also merge this if you like to.

mdix commented 8 years ago

So?

adammark commented 8 years ago

This looks good. I don't think we need to bump the version number, as you've only made changes in /extras. Can you revert package.json and markup.min.js, then I'll merge? Thanks!