-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Passive Node is not crawl-able link Pagespeed flag! #369
Comments
I’m hesitant to add The proper fix for this is probably to change to HTML tag used to I’m not sure if you want to add This also only affects the use of |
I recently ran into the same thing. Instead of adding href="#" adding role="none" may be a safer solution and should keep pagespeed quiet. |
I'll admit that I've not had time to properly research alternatives, so thanks for chiming in with |
What are you trying to do?
When using a passive node for structural distinction, google page speed flags that anchor element as "not crawlable link" and hurts the SEO score awarded to page because there is no href on an anchor tag.
What's your proposed solution?
The passive node should default with href="#" automatically, and possibly aria-hidden=true attribute. The href="#" solves the Pagespeed issue.
Additional context
While the passive node is a good designation for labels and headers on navigations, especially footers, it should probably be a tag instead, but understandably, there are probably a lot of legacy anchor tag styles involved.
#183
The text was updated successfully, but these errors were encountered: