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

Please add auto scroll #43

Open
shubich opened this issue May 9, 2018 · 5 comments
Open

Please add auto scroll #43

shubich opened this issue May 9, 2018 · 5 comments

Comments

@shubich
Copy link

shubich commented May 9, 2018

When a list is big and you navigating by arrows active element should be visible
I propose you to use scrollIntoView

activeElement.scrollIntoView(false);
@charliekassel
Copy link
Owner

That is an interesting method that I'd not seen before.
Although MDN says it's experimental - caniuse shows fairly decent support.
There is an open PR for this case - #37
@richardkmichael what do you think about this?

@richardkmichael
Copy link
Contributor

@charliekassel Funny timing.. I just discovered scrollIntoView() myself a few days ago, and was considering rewriting #37 again, because certainly it's much simpler. :-) However, MDN indicates Safari doesn't support the options object. I was going to experiment to be sure it isn't necessary.

@charliekassel
Copy link
Owner

I would imagine that the Boolean param is all we need.

@richardkmichael
Copy link
Contributor

Seems likely, yes. I'll see if I can get to it today, might be the weekend.

@danimalweb
Copy link

I would love to see this happen.

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

No branches or pull requests

4 participants