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

Dailymotion filter clobbers HTML #110

Closed
subimage opened this issue Apr 18, 2014 · 0 comments
Closed

Dailymotion filter clobbers HTML #110

subimage opened this issue Apr 18, 2014 · 0 comments

Comments

@subimage
Copy link

This seems to be an issue with lots of the filters - but DailyMotion filter specifically is clobbering HTML embeds. For instance, if you already have the HTML embed and run auto_html on it, the code tries to embed it inside the embed - breaking HTML.

I believe the default behavior should be to ignore all URLs already encased within a HTML tag.

PS: I fixed a similar issue with the image filter here https://github.com/subimage/auto_html/commit/110c62727a9cc30bcb4a11c7a58d8f462dfa58c0

@dejan dejan closed this as completed Dec 19, 2015
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