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.

Sign up for GitHub

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

Jump to bottom

Use replace rather than trim #133

Open

ironsidevsquincy wants to merge 1 commit into ded:master from ironsidevsquincy:master

ironsidevsquincy commented Aug 15, 2014

Browser support is IE9+, but this would go towards making it more lenient to older browsers that don't support trim


          use replace rather than trim (greater browser support)

c428fae

ironsidevsquincy mentioned this pull request

Greater browser support guardian/frontend#5599

Merged

sach2211 commented Aug 10, 2016

Would suggest using a better regex
.replace(/^[\s\uFEFF\xA0]+|[\s\uFEFF\xA0]+$/g, '')`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet