26408 - Can't use cursor to set focus in additional addressing field

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P3)

Description

[sol]

In the addressing widget of the Compose window, you can't use the cursor to set 
focus in an additional addressing field. Instead, you must hit <enter> to bring 
the focus to the next addressing field.

Build ID: 2000-01-08

Steps to reproduce:
1. Launch Mail
2. Open a Compose Window (on 3-pane toolbar, hit "New Msg" button)
3. Type email address in first line of addressing widget (*don't* hit <return>)
4. Position the cursor where the second line of the addressing widget should be 
and (left) click the mouse

Expected result: A second address entry field is drawn in the addressing widget 
below the first address entry field
Actual result: Nothing happens - you must get focus back into the first line of 
the addressing widget and hit <Enter> to get another addressing line

Comment 1

[Jean-Francois Ducarroz]

Accepting. 

Comment 2

[Jean-Francois Ducarroz]

done.

Comment 3

[Jean-Francois Ducarroz]

I've just found a small regression. When several recipient are prefiled (mailto, replay all), you cannot set the focus 
on the second recipient. I already have a fix for that in my tree.

Comment 4

[Jean-Francois Ducarroz]

here is the fix:

Index: addressingWidgetOverlay.js
===================================================================
RCS file: /cvsroot/mozilla/mailnews/compose/resources/content/addressingWidgetOverlay.js,v
retrieving revision 1.9
diff -r1.9 addressingWidgetOverlay.js
103a104,105
> 		
> 		setTimeout("awFinishCopyNodes();", 0);
504c506
< 
---
> 	
512a515,521
> }
> 
> 
> function awFinishCopyNodes()
> {
>     for ( var row = 2; row <= top.MAX_RECIPIENTS; row++ )
>         awFinishCopyNode(awGetTreeRow(row));

Comment 5

[Jean-Francois Ducarroz]

regression fixed.

Comment 6

[esther]

Using build 2000-03-01-09 on win98 and 2000-03-01-08 on linux and mac this 
is fixed as specifically stated in scenario: clicking mouse in place where next 
field should be,  will draw the field.  Note: this is only good for the 2nd and 
3rd field as that is all that currently displays in the addressing window and 
the populated fields don't automatically scroll up displaying the next blank 
field. I will look for or log a new bug for that bug.  Also note another known 
bug that shows up while testing this is the addressing field jumps up into the 
toolbar.  This bug is verified.