166164 - default for MDN confirmation dialog should be No (i.e. Do Not Send)

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect, P5)

Description

[timeless]

2002090108

I'm reading through 6000 messages by pressing <space>
99.9% of them are bugmails. a few are spams. some of the spams have requested
return receipts.

I expect that pressing space will safely lead me through my mail. The default
for MDN appears to be to send a confirmation message.  This is wrong and
dangerous. what should happen is that space safely refuses to send a
confirmation, and when I realize that I actually want to confirm it, I go back
and send the confirmation.

Comment 1

[grylchan]

Are you saying that you when you read a new mesg, you hit
space bar to advance the next new mesg, you are not
getting the confirmation dialog "The sender of the message has asked to
be notified when you read this message. Do you wish to notify the
sender?" 

By default it's set to 'ask me'. Are you saying by default
it should be 'don't send'? 

We don't actually send conrimation mesg until you you say ok or cancel.

Comment 2

[Ben Bucksch (:BenB)]

I think the prblem is that the default of that dialog is OK, so when you hit
space, you send the notification.

Comment 3

[timeless]

we should always ask. i should not be able to enable noninteractive MDN replies.
and yes if there's a prompt the default should be don't send.

Comment 4

[Henrik Gemal]

dont really think we can do anything here? It's a confirm dialog. Isn't Ok
always the default for dialogs like that?
You have to press ESC

Comment 5

[Ben Bucksch (:BenB)]

IIRC, you can usually specify the default in messageboxes. If XUL's msgbox
can't, then that's a bug.

Comment 6

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

Comment 7

[David :Bienvenu]

Comment on attachment 329301 [details] [diff] [review]
proposed fix

thx, Magnus.

Comment 8

[Magnus Melin [:mkmelin]]

Checking in mailnews/extensions/mdn/src/nsMsgMdnGenerator.cpp;
/cvsroot/mozilla/mailnews/extensions/mdn/src/nsMsgMdnGenerator.cpp,v  <--  nsMsgMdnGenerator.cpp
new revision: 1.34; previous revision: 1.33
done

->FIXED