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

report(description): update helpText for document.title audit #4677

Merged
merged 1 commit into from
Mar 2, 2018

Conversation

kaycebasques
Copy link
Contributor

Since the audit occurs in both the a11y and SEO categories, the new helpText now mentions both. The helpText links to a new reference on d.g.c/web that discusses both the a11y and SEO aspects of the audit. The previously-linked-to aXe doc only mentioned the a11y side. But that aXe doc is itself mentioned in the new reference. I also updated the failureDescription while I was here, since we agreed in group chat that contractions are more in-tune with the Lighthouse voice.

#polish

Since the audit occurs in both the a11y and SEO categories, the new helpText now mentions both. The helpText links to a new reference on d.g.c/web that discusses both the a11y and SEO aspects of the audit. The old aXe doc previously linked to only mentioned the a11y side.
@rviscomi
Copy link
Member

rviscomi commented Mar 2, 2018

LGTM 👍
blocked on google/WebFundamentals#5855

@paulirish paulirish changed the title update helpText for title audit report(description): update helpText for document.title audit Mar 2, 2018
@paulirish paulirish merged commit c60efa4 into master Mar 2, 2018
@paulirish paulirish deleted the docs-title branch March 2, 2018 21:12
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

Successfully merging this pull request may close these issues.

4 participants