207291 - [mac only] Unable to delete an email address in the address area

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P2)

Description

[Ninoschka Baca]

Trunk build 2003-05-23: Mac 10.1.5

Overview: Try to delete an email address in the address area and it fails. Mac
only issue.

Steps to reproduce:
1. Create a new message
2. Enter an email address in the address area 
3. Try and delete the email address by using the Delete key or highlighting the
address and then pressing the Delete key.

Actual Results: It fails to delete the address. 

Samir mentioned this problem occuring for the second entry but I can reproduce
the problem on the first entry. Also, the deletion fails whether AutoComplete is
on or off so it's a general delete problem.

Expected Results: The address should be deleted

Additional Information:
- 2003-05-23: WinMe, ok.
- 2003-05-23: Linux RH 8, ok.

Comment 1

[Ninoschka Baca]

Marking nsbeta1. This was working in the 5/22 build but regressed in the 5/23
build. 

Comment 2

[sairuh (rarely reading bugmail)]

shuehan, could the cause for bug 207260 be the culprit here, too? reassign as
needed!

Comment 3

[Greg K.]

This isn't critical. Minor at best, since there's a workaround (click the
Address button).

Severity is the wrong field to indicate priority, if that's what you meant.
That's what the Priority field is for.

Comment 4

[Simon Fraser [no longer active]]

This was pretty bad for me when I hit it today. If the delete key is
non-functional in the addressing widget, few people go around looking for
alternatives.

Comment 5

[Ninoschka Baca]

I think this is a 1.4 blocker.

Comment 6

[(not reading, please use [e-mail] instead)]

yes, I agree with ninoschka and simon, this is bad.  

a 1.4 final blocker.  I think p2.

for mail, something like this to the compose window is very painful.

I'll look at the checkins and see if anything jumps out.

Comment 7

[(not reading, please use [e-mail] instead)]

I have no proof yet, so don't take offense if I'm wrong, but I think this might
be caused by brade's checkin for:

http://bugzilla.mozilla.org/show_bug.cgi?id=184549
(mac: space no longer activates buttons in dialogs)

see
http://lxr.mozilla.org/mozilla/source/mailnews/compose/resources/content/addressingWidgetOverlay.js#866

we handle the delete with our own onkeydown handler.

see
http://lxr.mozilla.org/mozilla/source/mailnews/compose/resources/content/addressingWidgetOverlay.xul#31

Comment 8

[(not reading, please use [e-mail] instead)]

brade tells me this might be fixed by the fix for bug #206917

Comment 9

[Samir Gehani]

adt: nsbeta1+/adt1

If bug 206917 indeed fixes this please make sure the patch lands on the 1.4
branch and resolve this bug fixed with the fixed1.4 keyword.

Comment 10

[(not reading, please use [e-mail] instead)]

I'll land the fix on the branch, but since it is bryner's fix, giving credit
back to him.

Comment 11

[(not reading, please use [e-mail] instead)]

fix landed on the branch.  bryner landed on the trunk already.

Comment 12

[Greg K.]

Verified fixed using FizzillaMach/2003-05-29-08-trunk.

Comment 13

[sairuh (rarely reading bugmail)]

and vrfy'd on the branch (2003.05.29.05-1.4 commercial, on 10.2.6).