Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-autocomplete in md-dialog have huge height also with only few values #5826

Closed
mrigo opened this issue Nov 19, 2015 · 5 comments
Closed

md-autocomplete in md-dialog have huge height also with only few values #5826

mrigo opened this issue Nov 19, 2015 · 5 comments

Comments

@mrigo
Copy link

mrigo commented Nov 19, 2015

Hi,
I'm using md-autocomplete inside md-dialog and the md-virtual-repeat-container that shows the possible choises is too high when I have only one element.
It's the right behaviour?

Thanks
schermata 2015-11-19 alle 12 50 32

@sch4lly
Copy link

sch4lly commented Nov 19, 2015

+1, having the same "problem".

@jb29
Copy link

jb29 commented Nov 20, 2015

+1, I experienced the same issue on a customer's pc who's using IE11.
But unfortunately I couldn't reproduce the bug on my developer machine so far.

@elyas-bhy
Copy link

I reproduced this issue on this codepen.
This is basically a fork of the current demo of md-autcomplete (1.0.0-rc5), I just trimmed down the number of states to the first three.

You will also notice that clicking on Alabama, clearing the search text and clicking on the search bar again resizes the container height.

@gbaj
Copy link
Contributor

gbaj commented Dec 4, 2015

Min repo-

click the autocomplete box and type 'q'

http://codepen.io/anon/pen/KVKVjR

@Rid
Copy link

Rid commented Dec 4, 2015

I've also been having this issue, reversing the changes made to src/components/virtualRepeat/virtual-repeater.js in d6d7b08 seems to make the issue go away.

robertmesserle pushed a commit that referenced this issue Dec 8, 2015
robertmesserle pushed a commit that referenced this issue Dec 8, 2015
robertmesserle pushed a commit that referenced this issue Dec 8, 2015
robertmesserle pushed a commit that referenced this issue Dec 8, 2015
robertmesserle pushed a commit that referenced this issue Dec 9, 2015
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 a pull request may close this issue.

6 participants