afcapel / stimulus-autocomplete

Stimulus autocomplete component
MIT License
478 stars 61 forks source link

Set scrollIntoView behavior to "auto" to prevent lag when scrolling list #122

Closed duffyjp closed 1 year ago

duffyjp commented 1 year ago

I've added stimulus-autocomplete in an app with potentially large result sets. The smooth scrollIntoView in that case isn't a good fit. Your example app is on version 2 of stimulus-autocomplete which doesn't have that behavior. I have an update for that too I'll fire off a PR for in a moment.

Smooth Auto
smooth auto
smooth_2 auto_2
netlify[bot] commented 1 year ago

Deploy Preview for stimulus-autocomplete canceled.

Name Link
Latest commit 96675a7a060ac6e8c1db87ba11f6885f0f60d368
Latest deploy log https://app.netlify.com/sites/stimulus-autocomplete/deploys/63584bb247be7600084c985a
afcapel commented 1 year ago

Thanks @duffyjp, nice one!