matchboxdesigngroup / kindling

A WordPress theme that supports the Full Site Editing features.
MIT License
3 stars 0 forks source link

fix: update anchor link scroll-to positioning #24

Closed sstruemph closed 8 months ago

sstruemph commented 11 months ago

Changes proposed in this pull request

There's a script to handle smooth scroll of Anchor links. However it is set to scroll to the vertical center of the container with the anchor link. This sets it to scroll to the start or top of the container.

Closes https://github.com/matchboxdesigngroup/kindling/issues/23

Pre-submit checklist

As the author of this pull request, I verify that:

Testing

How to test the changes in this pull request

Follow the steps below to test the changes in this PR.

  1. Deploy this branch to the dev environment.
  2. Navigate to...

Functional tests

As the functional tester for this pull request, I verify that:

Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana. If all tests pass, move the task to "Ready for Code Review" in Asana and tag a team member for code review.

Code review

As the code reviewer for this pull request, I verify that:

Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana or continue with the "merging" steps below.

Merging

As the individual merging this pull request, I verify that:

┆Issue is synchronized with this Asana task