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

paper-autocomplete searchText is overriden on input #1043

Closed
Bartheleway opened this issue Jan 9, 2019 · 4 comments · Fixed by #1076
Closed

paper-autocomplete searchText is overriden on input #1043

Bartheleway opened this issue Jan 9, 2019 · 4 comments · Fixed by #1076

Comments

@Bartheleway
Copy link

Hi,

I'm using the latest v1.0.0-beta21 and I have an issue with autocomplete.

Whenever I type something in the autocomplete, it doesn't show me what I type. The second character override the first one so the search is always using 1 character ...

Does anyone got that one ?

Using Ember 3.7 (and 3.6 before).

@rahsheen
Copy link

I just noticed I was having the same issue, but it was due to trying to mut a relationship. So instead of something like @onSelectionChange={{action (mut this.fooBar)}} I was doing @onSelectionChange={{action (mut this.foo.bar)}}.

@xtagon
Copy link
Contributor

xtagon commented Apr 1, 2019

Running into this same issue, also with mut on a relationship. Is there a way to get around this?

@xtagon
Copy link
Contributor

xtagon commented Apr 1, 2019

This is happening for me on beta24, but I also tried a few older versions and have the same problem. So I'm wondering if it is a problem with the Ember version instead. I'm on Ember 3.8.

I don't think it's related to onSelectionChange either though, because I tried setting that to a custom action and even when it's not changing the selection, the bug with the search field overriding is exhibited.

Any thoughts?

@xtagon
Copy link
Contributor

xtagon commented Apr 1, 2019

Tried with more Ember + Ember-Paper versions. Pinning Ember-Paper to 1.0.0-beta.19 seems to fix it, but 1.0.0-beta.20 breaks.

That leads me to believe this is what broke it:

9503337

xtagon added a commit to xtagon/ember-paper that referenced this issue Apr 4, 2019
backspace added a commit to backspace/ember-paper that referenced this issue Jul 21, 2019
This reverts miguelcobain#1076, though then miguelcobain#1043 is surely a problem again.
backspace added a commit to backspace/ember-paper that referenced this issue Jul 21, 2019
This shows the bug being described in miguelcobain#1092. Reverting
miguelcobain#1076 does fix it, but then presumably miguelcobain#1043 would return 😞
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

Successfully merging a pull request may close this issue.

3 participants