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

Future handling of not supported HTML tags #63

Open
aileen opened this issue Aug 15, 2016 · 4 comments
Open

Future handling of not supported HTML tags #63

aileen opened this issue Aug 15, 2016 · 4 comments

Comments

@aileen
Copy link
Collaborator

aileen commented Aug 15, 2016

Hi @jbhannah,

at the moment, we're only transforming a few HTML tags into AMP HTML. I would love to add more transformations (e. g. audio and video) in the near future. Nevertheless, some tags will still remain prohibited by AMP.

Here's the full list of restricted HTML tags by AMP: https://github.com/ampproject/amphtml/blob/master/spec/amp-html-format.md#html-tags

I would like to know, if you are planning to strip out prohibited HTML tags, so the output will be a 100% valid AMP HTML? Maybe with the help of the Sanitize-HTML module?

Would love to hear your opinion about that!

Cheers, Aileen

@jbhannah
Copy link
Owner

Yes, ideally that will be one thing that this package does. For v1.0.0, its output should be fully valid AMP HTML.

@kirrg001
Copy link
Collaborator

amp twitter might be interesting as well.

@wtubog
Copy link

wtubog commented Jul 13, 2017

Any updates on this?

@Luen
Copy link

Luen commented Apr 11, 2020

I would really like to see support for the

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

No branches or pull requests

5 participants