Skip to content
This repository was archived by the owner on Jul 26, 2022. It is now read-only.

Conversation

@raymondmuller
Copy link

Got this issue that you can't preset the value. This fixes it.

Demonstrated in this pen: https://codepen.io/raymondmuller/pen/pxgXzd

the value of 'query' is 'test', however the input does not reflect it.

also watching the value afterwards to make it respond to external changes

@coveralls
Copy link

Pull Request Test Coverage Report for Build 39

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 69.697%

Totals Coverage Status
Change from base Build 38: 0.3%
Covered Lines: 48
Relevant Lines: 69

💛 - Coveralls

@Centro1993
Copy link

Please implement ASAP, much needed!

@brweber2
Copy link

Please accept and release new minor version. Thanks!

@rlightner
Copy link

Please accept this PR!

@siamkreative
Copy link

Can this get merged?

@dallasbeek
Copy link

dallasbeek commented Mar 12, 2019

ran into issue when binding the autocomplete in a vue for loop and dynamically adding a new record to my array on button click. Changing the below corrected.

inputValue: this.value
to
inputValue: this.value || ''

@siamkreative
Copy link

@alexurquhart Do you have time to review this PR?

jhalag added a commit to jhalag/vue-bootstrap-typeahead that referenced this pull request Apr 16, 2019
@mattzollinhofer
Copy link

Thanks for looking into this and submitting!

I merged in functionality that I think meets this need here: mattzollinhofer@e26ad49.

You can use the functionality under the forked project: https://www.npmjs.com/package/vue-typeahead-bootstrap or https://github.com/mattzollinhofer/vue-typeahead-bootstrap.

Hope this helps anyone who needs this functionality going forward! Let me know if that fix doesn't fix this issue.

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.

8 participants