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

Clean HTML #1

Merged
merged 1 commit into from May 23, 2015

Conversation

Projects
None yet
3 participants
@ambs
Contributor

ambs commented May 23, 2015

Very basic method to remove tags (comments, start tags and end tags).
Their content is preserved for now (unless for comments, of course).
Added a simple test file to test if tags are removed, and if
other less than are preserved. Hopefully this would work for most cases.
In the future a table of tags and behaviours might be relevant (for each
tag, to know if its content should be kept or removed, etc.)

Clean HTML
Very basic method to remove tags (comments, start tags and end tags).
Their content is preserved for now (unless for comments, of course).
Added a simple test file to test if tags are removed, and if
other less than are preserved. Hopefully this would work for most cases.
In the future a table of tags and behaviours might be relevant (for each
tag, to know if its content should be kept or removed, etc.)

bricas added a commit that referenced this pull request May 23, 2015

@bricas bricas merged commit 6cdd73a into edsu:master May 23, 2015

@ambs

This comment has been minimized.

Show comment
Hide comment
@ambs

ambs May 23, 2015

Contributor

Wow, that was fast! Thanks

Contributor

ambs commented May 23, 2015

Wow, that was fast! Thanks

@edsu

This comment has been minimized.

Show comment
Hide comment
@edsu

edsu May 23, 2015

Owner

👍

Owner

edsu commented May 23, 2015

👍

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