22338 - Can't remove attachment via Delete, hard to discove menu item

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P3)

Description

[Richard Zach]

An file, once attached to an outgoing message, can't be removed.

To reproduce:
1. New Msg
2. Attach a file
3. Select the file name in the attachments box
4. Press delete

Actual results: The character to the right of wherever the cursor is, is deleted
Expected results: The eattachment is removed

Linux build 1999.12.20.12

Comment 1

[pmock]

There is a work around that user can do.  If you want to remove a file
attachment from the mail compose window, you want to do the following:
 1) Highlight the desired attachment (url) in the attachment area.
 2) Select the File menu and choose Attach->Remove File Attachment
    The attachment url will be removed.

I checked this work around works on the commercial seamonkey build linux
1999-12-21-08-m13, win32 1999-12-21-09-m13, and macos 1999-12-21-08-m13.

The work around should also work in M12.

Comment 2

[Phil Peterson]

Workaround is ok for B1. Marking M15

Comment 3

[Jean-Francois Ducarroz]

comment from bug 26782:
------- Additional Comments From [e-mail]  2000-02-29 16:12 -------
Hey guys, if you put a tree in a toolbar, and don't explicitly override the 
user-focus CSS rule for toolbars (user-focus:ignore) then the tree won't get 
focus. IE no key events. Set the user-focus style to "normal".

I tried this with the attachment tree in mail compose, and this fixed the 
problem.

Tossing back to syd.

------- Additional Comments From [e-mail]  2000-02-29 16:30 -------
Also, Jean-Francois, can you add the style over-ride for the compose attachment
pane and check it in under this bug so we can at least fix that part?

Comment 4

[Jean-Francois Ducarroz]

The side effect of doing this is that I cannot tab anymore from the subject to the body! else, works much better.

Comment 5

[selmer (gone)]

Mass moving to M16 to get these off the M15 radar.  Please let me know if this
is really an M15 stopper.

Comment 6

[selmer (gone)]

Oops.  Really marking M18 now...

Comment 7

[Jean-Francois Ducarroz]

Fixed and checked in. Now you can remove aan attachment by using either the 
backspace key or the delete key. The menu item "Remove Attachment" has been 
removed.

Comment 8

[pmock]

Verified as fixed on win32, macos, and linux using the following builds:
 win32 commercial seamonkey build 00042609-m16 installed on Dell P500, Win98
 macos mozilla seamonkey build 00042611-m16 installed on G3/400, MacOS 9
 linux commercial seamonkey build 00042608-m16 installed on P200, RedHat 6.1