81296 - File|Send Now menu item is always enabled.

Status: VERIFIED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect, P3)

Description

[Ninoschka Baca]

Build 2001-05-16-04: WinMe
Build 2001-05-16-08: Mac 9.04


Overview: In a new message/compose window the File|Send Now menu item is 
disabled. This happens using classic/modern, plain/html. 

Expected Results: I can send using the mouse to select the Send button but it's 
not possible to use the keyboard to select this item (i.e. arrowing down to the 
menu item, mnemonics --- currently no accelerator exists which is a known 
issue). 

Also important for accessibility reasons.

Comment 1

[Ninoschka Baca]

Marking nsbeta1 to have a way to send messages without requiring a mouse.

Comment 2

[scottputterman]

reassigning to varada and cc'ing prass (since she has worked on other compose
enabling/disabling bugs)

Comment 3

[Ninoschka Baca]

Additional Information:
- Build 2001-05-17-08: Linux, the problem also occurs.

Comment 4

[Marek Z. Jeziorek]

setting TM to 0.9.2 per PDT triage (however, you can check it in into 0.9.1 by
18/May/01 11:59pm or into 0.9.2 trunk when it opens)

Comment 5

[Håkan Waara]

*** Bug 83513 has been marked as a duplicate of this bug. ***

Comment 6

[Håkan Waara]

*** Bug 83512 has been marked as a duplicate of this bug. ***

Comment 7

[Christopher Blizzard (:blizzard)]

Seth, you said that you wanted this bug.  Take it if you do.

Comment 8

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

I'll leave it for varada, as he's the proper owner.

currently the code updates that command when the offline state changes, but 
that's not right.  (for one, what if we are offline when the compose window 
comes up?  the state would be wrong.)

don't we have the means now [inspired by mscott's work to the 3 pane, and then 
hyatt made it work everywhere with command="" or observes=""] to check a 
command is enabled when the menu is created?

Comment 9

[Christopher Blizzard (:blizzard)]

Any luck finding this yet?

Comment 10

[varada]

The Send Now  was not getting initially enabled because of a timing issue and
that has been taken care of. The remaining problem now is that even thou it is
disabled when offline the UI continues to keep it visually enabled and not
greyed out. Still investigating.

Comment 11

[scottputterman]

adding PDT+

Comment 12

[John Gaunt (redfive)]

using the beta candidate from 6/7 I am seeing this still. I compose a message
and try to send via the menu and Send Now is grayed out. Sending using the icon
on the tool bar works fine.

win2k

Comment 13

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

you're still seeing it because it is not fixed.

Comment 14

[varada]

Attaching a patch.
This doesnt solve the problem -merely makes it more tolerable. Instead of having
the SendNow be disabled when online,it is now permanently enabled (visually). I
say visually because when offline thou it seems enabled you still cant send now
but use only send later.

Comment 15

[varada]

Attachment: patch v1

Comment 16

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

this is probably for the best for now.

sr=sspitzer

better to have it always enabled until we can figure the proper fix.

after you check in, please log a new bug on File | Send Now always being enabled.

Comment 17

[Christopher Blizzard (:blizzard)]

r=blizzard

Comment 18

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 19

[varada]

Partial fix checked in -File|Send Now is always enabled now (only visually
though functionality remains consistent).
Changing summary to reflect the new condition.
PDT should re-evaluate this bug now.

Comment 20

[scottputterman]

That's great.  Moving to future milestone.

Comment 21

[R.K.Aa.]

*** Bug 124562 has been marked as a duplicate of this bug. ***

Comment 22

[Calvin Liu]

Why don't we change the status of this bug into RESOLVED? Are we still waiting 
for any comment or better patch?

Comment 23

[varada]

This should be marked WFM because it doesnt happen anymore.

Comment 24

[Olga]

Verified on trunk 10/17/02, Win2K.