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

community page amp error: The tag 'img' should be replaced with an equivalent 'amp-img' tag. #180

Open
fifieldt opened this issue Dec 13, 2020 · 4 comments

Comments

@fifieldt
Copy link
Collaborator

image

@matrinox
Copy link
Member

This isn't straightforward to fix. Either the whole site needs to be AMP-compatible or we just ignore this one page, the only page with images. But making it AMP-compatible has its own problems, like the FB chat bot we added, search, and more. It's very restrictive.

Another option we can try is to have 2 versions of the community page (AMP and non-AMP), but that would make maintaining that page more difficult down the road.

What would be the more preferable solution? Ignore or duplicate?

@erickhun
Copy link
Collaborator

@matrinox the effort doesn't worth the reward. I'd ignore it

@fifieldt
Copy link
Collaborator Author

I saw an article recently where Google was talking about removing the priority given to AMP sites anyway. Did that apply only to news, or is that something we should consider too?

@matrinox
Copy link
Member

From my understanding, they've always said they've prioritized on speed, not AMP; AMP is just one way to create fast loading pages

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

3 participants