Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove RegExp.escape #116

Closed
benjamingr opened this issue Oct 2, 2015 · 1 comment
Closed

Remove RegExp.escape #116

benjamingr opened this issue Oct 2, 2015 · 1 comment

Comments

@benjamingr
Copy link
Contributor

Sadly, two months ago my proposal has been rejected by the TC.

The good part is that there is work on a more extensive proposal using a tag that is completely safe.

In the meanwhile, it might best to remove .escape from the polyfill.

@zloirock
Copy link
Owner

zloirock commented Oct 2, 2015

Yes, thanks, I know it.

I think, it should be available as minimum before adding an alternative of this method. It will be moved to the non-standard (core) namespace. It's breaking change and I don't know when it will be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants