Skip to content
This repository has been archived by the owner on Aug 10, 2022. It is now read-only.

[ligthouse] add "descriptive link text" reference #5813

Merged
merged 2 commits into from
Feb 23, 2018
Merged

Conversation

kaycebasques
Copy link
Contributor

greetings @rviscomi ... got another one for ya

What's changed, or what was fixed?

  • add reference for "links do not have descriptive text" audit

Fixes: N/A

Target Live Date: 2018-02-28

  • This has been reviewed and approved by @rviscomi
  • I have run gulp test locally and all tests pass.
  • I have added the appropriate type-something label.
  • I've staged the site and manually verified that my content displays correctly.

CC: @petele

@WebFundBot
Copy link

👍

@petele
Copy link
Member

petele commented Feb 22, 2018

@rviscomi - if you approve, please feel free to "Squash and Merge"


In general, the link text should clearly indicate to users what type of content they'll get if
they click the link. Using the exact title of the document, or a description of the page, is
usually good enough. Avoid using URLs as link descriptions.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/link/anchor/ for consistency with the overview above?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, I thought about this too, but opted to use both terms to signify that they are interchangeable, and improve our own SEO

they click the link. Using the exact title of the document, or a description of the page, is
usually good enough. Avoid using URLs as link descriptions.

[style]: https://developers.google.com/style/link-text
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this used anywhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, good catch, thanks

{# wf_published_on: 2018-02-22 #}
{# wf_blink_components: N/A #}

# Links Do Not Have Descriptive Text {: .page-title }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How much do you think these docs should go into the SEO benefits of [doing the thing the audit checks for]? Linking to relevant SEO docs is good too.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about that, but I haven't really seen anything else on it, other than "it helps search engines understand your content"

We could link to webmaster blog, to give it authority, but the content is pretty much duplicated

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you have specific link and specific content to add, that'll help me see what you're thinking

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here's one: https://support.google.com/webmasters/answer/7451184#optimize (scroll down to "Write good link text" section, no closer anchor to jump to... grr)

@WebFundBot
Copy link

👍

@kaycebasques kaycebasques merged commit bbbc032 into master Feb 23, 2018
@kaycebasques kaycebasques deleted the lh-anchors branch February 23, 2018 16:26
@WebFundBot
Copy link

🎉 This has been pushed live to https://developers.google.com/web/

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

Successfully merging this pull request may close these issues.

5 participants