Skip to content
This repository has been archived by the owner on Apr 3, 2021. It is now read-only.

goToOffset + Keyboard direction #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jgi-comspace
Copy link

Hi!
This PR contains two small changes/fixes:

  1. The offset for "scrollThere" in .goTo() was hard coded to -100, which isn't always useful. The option goToOffset was added and documented in the readme to address this. The default value is still -100 to ensure backwards compatibility.
  2. When using the Keyboard navigation, no directions were handed over to "goTo()" – this could be easily fixed, since the direction is known at that point.

cheers,
John Gibbon

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

Successfully merging this pull request may close these issues.

1 participant