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

Add tests for normalize outer whitespace removal #246

Closed

Conversation

Projects
None yet
2 participants
@cirosantilli
Copy link
Contributor

commented Dec 19, 2014

  • expose failure with ' <p>a</p>' which gives <p>a</p>,
    '<p>a</p>' expected
  • rephrase it slightly to mention that it strips inner spaces also as in <p>a </p> to <p>a</p>

@cirosantilli cirosantilli force-pushed the cirosantilli:test-whitespace-strip branch 5 times, most recently from 8ac1dfa to 26fdd61 Dec 19, 2014

Add tests for normalize outer whitespace removal
-   expose failure with `'  <p>a</p>'` which retains one space before,
    `'<p>a</p>'` expected
-   rephrase it slightly to mention that it strips inner spaces also
    as in `<p>a </p>` to `<p>a</p>`

@cirosantilli cirosantilli force-pushed the cirosantilli:test-whitespace-strip branch from 26fdd61 to 8f14ebb Dec 19, 2014

@jgm jgm closed this in a8d97a0 Dec 23, 2014

@jgm

This comment has been minimized.

Copy link
Member

commented Dec 23, 2014

Thanks for this and the other PRs. I resolved this in a slightly different way, in the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.