fagnermartinsbrack/str-replace

A simple, lightweight JavaScript API to replace a String

str-replace's Issues

Replace all fail

Hi,

I used the text : ' value value ' and try to replace to text ' 1 ', the result is ' 1 value ', just the first item was replaced.

I use the version : "str_replace": "^0.1.6",

Mention in the docs that the library is a syntactic DSL syntactic sugar around RegExp

See #1 (comment).

Use `replace.first()` instead of the default `replace()` API

Since we have replace.all() it makes sense having a replace.first() instead of implicitly making replace() act as replacing the first occurrence by default, which is not self-documenting.

Feedback received on /r/node

Investigate the performance implications using regex for replace

Investigate and document if there is a considerable change in execution time for the major browsers using regex for the replace/replaceAll operations. The problem with using regex is that it will be a lot of work to escape the proper characters. But if it means a considerable benefit ... so be it.