Pass through extra arguments to scrollTo #2069

[nurdism]

This is a PR for issue #2069
Adds smooth scrolling option to scrollBehavior position.

[posva]

Thanks for the PR but It will be better to pass any extra arguments to build up an object, so the user can pass the original behaviour function. Also, it should be able to still call the old two parameters functions in browser not supporting it

[nurdism]

So your saying something like this or did I miss understand your comment?

[posva]

Yes, it's something like that

On Fri, Mar 9, 2018 at 11:21 AM Nurdism ***@***.***> wrote: So your saying something like this or did I miss understand your comment? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2096 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAoicW8bAiMnWDbuznHFSbrjLQtFJYX2ks5tcle2gaJpZM4Sj_Qm> .

-- Eduardo San Martin Morote

[nurdism]

I'm wondering if it would be better to wrap it with a try { } catch { } rather than the method I used. That way it would work on older browsers if you include the polyfill, thoughts?

[GustavoNunes]

Hi there! It would be nice if this PR gets merged, is something holding this back?

I can help if needed (was going to submit a PR but found this one).
Thanks!

[posva]

let's do

behavior:  (isObject && shouldScroll.behavior) || 'auto'

the user should be responsible for passing the correct type of value

[posva]

There is a problem with the repository so I will add the change locally