Screenshot attached. This also occurs in the Minnelli theme. It does not occur in Bluemarine.

Labelling as critical considering the magnitude of the offset, the nature of the browser and the proximity of D5's release.

Feel free to recategorise.

-K

CommentFileSizeAuthor
#8 autocomplete-pos.patch915 bytesSteven
#2 104537.png13.1 KBkeith.smith
ie-dropdown.png6.35 KBZen

Comments

Steven’s picture

Can someone test/verify this in IE7? It matters a lot to how we approach the solution.

keith.smith’s picture

StatusFileSize
new13.1 KB

See attached screen shot from IE7

Steven’s picture

I'll see what I can do. This problem is explained partially in this blog post btw:
http://acko.net/blog/mouse-handling-and-absolute-positions-in-javascript

jacauc’s picture

Title: Autocomplete drop down positioning issues in IE6 » Autocomplete drop down positioning issues in IE6 & IE7

Subscribing to this, and updating the title.

m3avrck’s picture

The latest jQuery version, 1.0.4 has support for pageX and pageY across *all* browsers now, IE included.

http://jquery.com/blog/2006/12/12/jquery-104/

This might help the problem, not to mention with so many nice bugfixes it'll be great to see it in core. I haven't played with it yet but throwing that out there as a possible solution.

Steven’s picture

Errr no. The jQuery pageX fix is just adding a scroll offset to clientX. It does not help at all, as the problem I had (in the blog entry) was relating these mouse coordinates to element positions. Usually this is accomplished by getting an element's absolute position, which turns out to be a difficult problem (as evidenced in this bug).

Quirksmode has a nice case study:
http://www.quirksmode.org/js/findpos.html

anders.fajerson’s picture

Does it have to be positioned with js, maybe CSS could be used instead? I did a testpage for a similair usercase a while ago, which used CSS to postion a DIV relative an input field: http://perifer.se/test/datepickerCSS.htm

Steven’s picture

Status: Active » Needs review
StatusFileSize
new915 bytes

Here is a patch which implements the suggested approach. The div is now placed before the input rather than on the top level, and positioned relative to it instead of absolute.

Tested in Safari, Opera and Firefox. Needs IE6 and 7 testing.

ChrisKennedy’s picture

Great improvement, patch completely fixes the problem in IE6.

heine’s picture

Status: Needs review » Reviewed & tested by the community

Fixes the problem in IE 7 as well.

Steven’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD, thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)