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

Unescape: Fix & improvement #6

Merged
merged 2 commits into from Feb 24, 2015

Conversation

Projects
None yet
4 participants
@floridoo
Contributor

floridoo commented Feb 24, 2015

  • replace all occurences, not just the first one
  • added additional html entities (e.g. hogan.js escapes them)
@kamilogorek

This comment has been minimized.

Show comment
Hide comment
@kamilogorek
Member

kamilogorek commented Feb 24, 2015

LGTM

latentflip added a commit that referenced this pull request Feb 24, 2015

@latentflip latentflip merged commit 61f61b9 into AmpersandJS:master Feb 24, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@bear bear added request bug labels Feb 24, 2015

@floridoo floridoo deleted the floridoo:unescapefix branch Feb 25, 2015

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