[#7074] address suggestion popup becomes unusable as you add more recipients
Summary address suggestion popup becomes unusable as you add more recipients
Queue IMP
Queue Version 4.2
Type Bug
State Resolved
Priority 2. Medium
Owners slusarz@horde.org
Requester liamr@umich.edu
Created 2008-07-16 (4083 days ago)
Due
Updated 2008-11-09 (3967 days ago)
Assigned 2008-08-22 (4046 days ago)
Resolved 2008-11-09 (3967 days ago)
Milestone 4.2.1
Patch No

Comments
liamr@umich.edu 2008-07-16 15:05:12
When composing a message using IE6 or IE7, the address look up 
suggestion pop-up starts to move left as you add recipients.   Pretty 
much by the time the field (To, Cc, Bcc) starts to fill up and start 
scrolling existing entries to the left, the pop up will move to the 
left as well, eventually moving beyond the boundaries of the 
composition window.

Chuck Hagenbuch <chuck@horde.org> 2008-07-16 18:15:26
Confirmed.

Michael Slusarz <slusarz@horde.org> 2008-07-22 04:57:56
This is fixed if using the latest dev version of prototypejs.  I 
glanced at the code, but there have been so many changes to the 
positioning functions that there is little/no chance we would be able 
to do a clean patch to 1.6.0.2.  Don't know if this bug is nasty 
enough to upgrade to the latest dev version (?)

Jan Schneider <jan@horde.org> 2008-07-23 15:28:09
I would say no, unless someone is coming up with a stripped-down patch 
the exactly fixes this issue.

Michael Slusarz <slusarz@horde.org> 2008-07-29 03:28:21
Then this is probably better marked as stalled, since the next version 
of prototypejs will fix this.

mark.manning@nexussafe.com 2008-08-20 16:23:46
We worked around the problem by adding code in the fixIEOverlapping 
function in imp/js/controls.js to calculate the proper absolute 
position for the autocompletion div.  This file has been stripped and 
renamed to autocomplete.js (as well as other changes to other files 
for other fixes) which is why we didn't submit this for use.  Anyway, 
here's the code if it's useful:



fixIEOverlapping: function() {

     Position.clone(this.update, this.iefix, 
{setTop:(!this.update.style.height)});

     this.iefix.style.zIndex = 1;

     this.update.style.zIndex = 2;

     //*** BEGIN Change ***

     // Need to set the left position bceause IE shifts it to the left 
for each added address

     var e = document.getElementById('to');

     var n = e.offsetLeft;

     var p = e.offsetParent;

     while(p&&p.tagName!='BODY')

     {

         n+=p.offsetLeft;

         p=p.offsetParent;

     }

     this.update.style.left = n;

     //*** END Change ***

     Element.show(this.iefix);

   }



Not elegant but it does work.  Sounds like the problem will go away in 
the near future anyway.



Michael Slusarz <slusarz@horde.org> 2008-08-22 06:01:59
Thinking about this more - this is *really* broken behavior.  I think 
this is blocking on IMP 4.2.1.  Given the hack solution given in the 
previous comment, we are better off using the latest pre-1.6.0.3 
version of prototypejs (especially since it provides some other fixes 
that are useful elsewhere).

CVS Commit <cvs@lists.horde.org> 2008-09-02 21:08:58

CVS Commit <cvs@lists.horde.org> 2008-09-02 21:12:11

Michael Slusarz <slusarz@horde.org> 2008-09-02 21:12:50
Figured out a workaround until prototypejs is updated.  Fixed for 4.2.1.

Chuck Hagenbuch <chuck@horde.org> 2008-11-09 16:16:06
We've updated to 1.6.0.3 final, so I'm closing this.