-
Notifications
You must be signed in to change notification settings - Fork 3.4k
fix(autocomplete): show dropdown on top only when there is room #11629
Conversation
1656e0e
to
0899e0e
Compare
OK, @marosoft here's the information that I have for when this new calculation doesn't work. The use case is an After this change to the calculation went in, the position is |
I wasn't able to reproduce this in testing yesterday with Today I just learned that this issue was happening with |
Note that |
The issue with the dropdown being displayed below the dialog has been tracked down. This was caused by custom CSS overrides conflicting with the updated positioning. The custom CSS can be adjusted. The remaining issue is a concern that the behavior is changing where sometimes the dropdown is displayed on |
@jelbourn caretaker note: please check with Miles to verify that the affected team updated their app before syncing this to google3. |
PR Checklist
Please check that your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The autocomplete options panel is shown using the
top
position even when there is no room to display the options. More details in #10859.Issue Number:
Related to #11626. Related to #11575. Fixes #10859.
What is the new behavior?
This reverts commit 46e08e5 which reverted the previous fix (a37f7cc) to this issue due to a regression.
Does this PR introduce a breaking change?
Other information
At this time, this PR still causes the regression and no reproduction steps have been defined to allow us to investigate or fix the regression.