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

Correcting header typo on scroll-tracking.js #752

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Conversation

vp2015
Copy link
Contributor

@vp2015 vp2015 commented Feb 29, 2016

Correcting typo on heading 'When you can claim Universal Credit' on /guidance/universal-credit-how-it-helps-you-into-work which prevented the scroll event from firing.

Correcting typo on heading 'When you can claim Universal Credit' on /guidance/universal-credit-how-it-helps-you-into-work which prevented the scroll event from firing
@erkde
Copy link
Contributor

erkde commented Feb 29, 2016

I wonder if in the future, it would be worth adding the links into the PR description, so we can click through and verify the target page exist, and that the headings exist.

For example:

URL on gov.uk

  • Add scroll tracking to heading "When you can claim Universal Credit"

Even though it doesn't verify the heading text is matching, it will make it easier for the reviewer of the PR to go to the target page and check

@vp2015
Copy link
Contributor Author

vp2015 commented Feb 29, 2016

Yep good idea, will do from now on

erkde pushed a commit that referenced this pull request Feb 29, 2016
Correcting header typo on scroll-tracking.js
@erkde erkde merged commit 8b886a2 into alphagov:master Feb 29, 2016
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.

2 participants