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

Fix link text for anchor links to GDS Way pages #1024

Merged
merged 1 commit into from
May 13, 2021

Conversation

lfdebrux
Copy link
Member

Commit 8d52920 updated the URL for links to style guides that had been moved to the GDS Way. In doing so the links now violate the rule that anchor links should not be used without explaining that the link goes into a page 1.

This commit updates the link text so that it is clear that the links take you to a place within a page.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1024 May 13, 2021 12:18 Inactive
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Contributor

@EoinShaughnessy EoinShaughnessy left a comment

Choose a reason for hiding this comment

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

Looks good! Just left 2 small comments.

Commit 8d52920 updated the URL for
links to style guides that had been moved to the GDS Way. In doing so
the links now violate the rule that anchor links should not be used
without explaining that the link goes into a page [1].

This commit updates the link text so that it is clear that the links
take you to a place within a page.

[1]: https://docs.google.com/document/d/1XY8RG2uQjQNo7PFhtjvqIcHqGqSgGDRuQGkpOFpc6iM/preview

Co-authored-by: Eoin Shaughnessy <eoin.shaughnessy@digital.cabinet-office.gov.uk>
@lfdebrux lfdebrux force-pushed the ldeb-fix-anchor-link-content branch from 86ba450 to fde89db Compare May 13, 2021 13:52
@lfdebrux
Copy link
Member Author

Looks good! Just left 2 small comments.

Thanks :)

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1024 May 13, 2021 13:53 Inactive
@lfdebrux lfdebrux merged commit ad865a4 into main May 13, 2021
@lfdebrux lfdebrux deleted the ldeb-fix-anchor-link-content branch May 13, 2021 14:01
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.

3 participants