search

afcapel / stimulus-autocomplete

Stimulus autocomplete component
MIT License
482 stars 62 forks source link

Delete removeAttribute('value') on input change #123

Closed seb-jean closed 1 year ago

seb-jean commented 2 years ago

It is not useful to remove the value attribute

netlify[bot] commented 2 years ago

Deploy Preview for stimulus-autocomplete canceled.

Name Link
Latest commit 2e4d4a71d94ab888c9ca7717bf13340d5f47d721
Latest deploy log https://app.netlify.com/sites/stimulus-autocomplete/deploys/6364f24cb26182000869ab2a
afcapel commented 1 year ago

Thanks @seb-jean, good point 👍

I've merged the change in bdb17534a6088c7aea6d6a4264ae159a203ca5d7 with a more explanatory commit message.