Skip to main content

Search and Top Navigation

#9680 closed bug (duplicate)

Opened November 27, 2013 03:53PM UTC

Closed July 24, 2014 07:36PM UTC

position.collision 'flip' does not work with autocomplete

Reported by: romario333 Owned by: romario333
Priority: minor Milestone: none
Component: ui.autocomplete Version: 1.10.3
Keywords: Cc:
Blocked by: Blocking:
Description

I set autocomplete's position collision option to 'flip'. It is not respected however, flipping just does not happen.

See this plunkr for the demo:

http://plnkr.co/edit/HRgLYZYDKg0PnoetVkUY?p=preview

I have tested this on several browsers (Chrome 31, Firefox 25 and IE 10) and they are all affected.

It seems to be caused by this commit:

7f808b2047725cd8fde51a948cb4e5f5946c82e1, Position: Check to also make sure to only flip if more of the element is inside "within"

If I revert this commit, 'flip' starts to work correctly.

Attachments (0)
Change History (5)

Changed November 27, 2013 04:01PM UTC by scottgonzalez comment:1

_comment0: That link is working just fine for me. Tested in Chrome.1385570578948771
_comment1: That link is working just fine for me. Tested in Chrome 30.1385570616537163
owner: → romario333
status: newpending

That link is working just fine for me; tested in Chrome 30.

Changed November 27, 2013 04:08PM UTC by romario333 comment:2

status: pendingnew

So suggestions were shown above the text inputs in your case? For both fields? (The right one is actually hacked to work correctly).

Changed November 27, 2013 04:13PM UTC by scottgonzalez comment:3

status: newopen

Now I see that there's a problem if the space gets small enough. It flips for a while, but then stops.

Changed November 27, 2013 04:45PM UTC by scottgonzalez comment:4

description: I set autocomplete's position collision option to 'flip'. It is not respected however, flipping just does not happen. \ \ See this plunkr for the demo: \ \ http://plnkr.co/edit/HRgLYZYDKg0PnoetVkUY?p=preview \ \ I have tested this on several browsers (Chrome 31, Firefox 25 and IE 10) and they are all affected. \ \ It seems to be caused by this commit: \ \ 7f808b2047725cd8fde51a948cb4e5f5946c82e1: kborchers <k_borchers@yahoo.com>, Position: Check to also make sure to only flip if more of the element is inside "within" \ \ If I revert this commit, 'flip' starts to work correctly.I set autocomplete's position collision option to 'flip'. It is not respected however, flipping just does not happen. \ \ See this plunkr for the demo: \ \ http://plnkr.co/edit/HRgLYZYDKg0PnoetVkUY?p=preview \ \ I have tested this on several browsers (Chrome 31, Firefox 25 and IE 10) and they are all affected. \ \ It seems to be caused by this commit: \ \ [https://github.com/jquery/jquery-ui/commit/7f808b2047725cd8fde51a948cb4e5f5946c82e1 7f808b2047725cd8fde51a948cb4e5f5946c82e1], Position: Check to also make sure to only flip if more of the element is inside "within" \ \ If I revert this commit, 'flip' starts to work correctly.

Changed July 24, 2014 07:36PM UTC by scottgonzalez comment:5

resolution: → duplicate
status: openclosed

Duplicate of #8710.