-
Notifications
You must be signed in to change notification settings - Fork 82
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
meta description #19
Comments
I could do this but I'm not sure of the added value. I suspect most sites that are reading this kind of meta from the HTML itself aren't running the page's JS—think e.g., sharing a link on Twitter. Is there another reason why this might be useful? |
The main reason is to avoid this Lighthouse's warning: https://developers.google.com/web/tools/lighthouse/audits/description |
I really believe that most crawlers/etc are not going to run page JavaScript just to read your site's meta tags. So while it might make Lighthouse happy, I suspect this change won't actually improve your SEO. |
I understand your point, but I'm not sure it would be a bad thing. You could just check if there is a meta description already and add one based on manifest if not. Just as |
Unlike
I haven't googled it but I'm quite sure Google can index script-generated content, however if Google saw the same Personally I don't think this is a good default for PWACompat. |
Thank you for having a look at it @samthor. I agree with you. |
It would be nice to read
description
from the manifest and add it as<meta name="description" />
if that tag is not already present.The text was updated successfully, but these errors were encountered: