md-autocomplete not firing focus events.

[dmackerman]

Angular 1.4.3 and Material 0.10.1-rc5.

I've copied the examples for mdAutocomplete directly in my application and am seeing some weird issues.

For whatever reason, when the field is focused - not having typed anything yet - no events (and therefore no classes) are added, so the menu doesn't show "all" results. In the docs, the md-menu-shown class is added to the md-autocomplete-wrap element on mouse focus.

This may be related to #4046, but I can't say for sure. This happens to me everywhere, not just confined to inside an mdDialog.

I can't seem to get it to reproduce on Codepen or in the docs application, so i'm not sure why this is happening. The autocomplete works as expected once keystrokes are entered, but it's pretty broken behavior to not show "all" results on focus if the entire array is available on scope.

[elubin]

Ditto. Weird focus events with md-autocomplete on an md-dialog inside md-chips. (although i'm not sure that has anything to do with it, but it makes seeing the issue easier) In my case, the issue is because md-min-length="0" because i wanted to show all 9 options to the user immediately instead of making them select one.

http://jsfiddle.net/5m9rpcrn/

(added the same note to 4046)

[robinsving]

@dmackerman I am seeing the same issues with md-autocomplete.
On initial focus, nothing is added.
Upon typing anything in the field the (filtered) list is shown, and when removing all input, then blurring and then refocusing it now shows the entire list as it should.
Console logging shows that the filtering function is not triggered on first focus
When using Codepen, however, the error is not reproducible - here it works as intended.

My angular-materials version comes from npm, version 0.10.1

[dmackerman]

Glad others are seeing this.

Edit: Any chance we can get an update from one of the Material folks on this?

[EdenCoder]

+1

[topherfangio]

I just pushed PR #4958 which fixes many issues and I will post an update here once it is merged asking if you can retest against that version.

[dmackerman]

Thanks @topherfangio. Will check it out again once it's merged.

[topherfangio]

@dmackerman PR #4958 has been merged, can you please test again?