Shutdown Service should so shutdown dialog non-modal

RESOLVED FIXED

Status

MailNews Core
Backend
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Nick Kreeger, Assigned: Nick Kreeger)

Tracking

Trunk
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 287706 [details] [diff] [review]
Patch V1

Since a shutdown task might need to prompt a user for a password (or some other type of data), the shutdown dialog should be displayed modal-less.
Attachment #287706 - Flags: review?(bienvenu)

Updated

10 years ago
Attachment #287706 - Flags: review?(bienvenu) → review+
(Assignee)

Comment 1

10 years ago
Checked into TRUNK.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Comment 2

10 years ago
Comment on attachment 287706 [details] [diff] [review]
Patch V1

>     mMsgProgress->OpenProgressDialog(internalDomWin, topMsgWindow, 
>                                      "chrome://messenger/content/shutdownWindow.xul", 
>-                                     showModal, nsnull);
>+                                     PR_FALSE, nsnull);
So, does this mean that the extra parameter to openProgressDialog is no longer necessary?
(Assignee)

Comment 3

10 years ago
(In reply to comment #2)
> (From update of attachment 287706 [details] [diff] [review])
> >     mMsgProgress->OpenProgressDialog(internalDomWin, topMsgWindow, 
> >                                      "chrome://messenger/content/shutdownWindow.xul", 
> >-                                     showModal, nsnull);
> >+                                     PR_FALSE, nsnull);
> So, does this mean that the extra parameter to openProgressDialog is no longer
> necessary?
> 

In theory yes - my first thought was that it wouldn't hurt to leave in. But I can easily change that back if you want Neil.

Comment 4

10 years ago
I was thinking it might be useful for other users...
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.