The "show dialog" setting should be a per-profile pref

NEW
Unassigned

Status

MailNews Core
Simple MAPI
16 years ago
10 years ago

People

(Reporter: Srilatha Moturi, Unassigned)

Tracking

Trunk
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

16 years ago
This bug was created off of 103908 based on comments from dveditz

> PRBool getShowDialog() {
>     PRBool rv = PR_FALSE;
>     nsCString showDialog(GetRegistryKey(HKEY_LOCAL_MACHINE, 
>                                         "Software\\Mozilla\\Desktop", 
>                                         "showMapiDialog"));

Why is this a global registry setting? The "show dialog" setting should be
a per-profile pref. I wouldn't want my wife to be bothered by this dialog,
but I'd want to see if some other app changed my settings.

This is regarding the The default email selection dialog box.
Jennifer,robinf what is your take on it.
This is for next release and not for 0.9.4

Comment 1

16 years ago
The way the Windows OS sets a mail app to be the default mail app is for the 
system.  Once it's set, it affects all windows accounts on the system.  So 
having the show dialog _not_ be a per-profile pref makes sense.
The setting is global, but some profiles may not care which way its set and
*really* hate nag dialogs. I wouldn't want my wife's setting "Do not show again"
to cause me to miss the fact that Outlook or other app stole my default.

Comment 3

16 years ago
Mark as P3
Priority: -- → P3

Comment 4

16 years ago
Too late RFE at this time. Let's try and get it next time = PDT- for 094.
Keywords: nsbranch
Whiteboard: [PDT-]

Updated

16 years ago
Blocks: 107065

Updated

16 years ago
Keywords: nsbranch

Updated

16 years ago
Target Milestone: --- → mozilla0.9.8
(Reporter)

Comment 5

16 years ago
I agree with Sean on this. That's how the windows integration dialog works too.
But the windows integration has an option "Alert me if some other app changes 
it". May be we can have some thing like that for this dialog too.
Jennifer, Robin what's your opinion.

Also, moving milestone to mozilla 1.1
Target Milestone: mozilla0.9.8 → mozilla1.1

Comment 6

16 years ago
removing PDT grafitti.
Whiteboard: [PDT-]

Updated

16 years ago
No longer blocks: 107065

Comment 7

16 years ago
removed tiantian from cc (bad alias; sends mail to jhooker)

Updated

15 years ago
QA Contact: trix → yulian
QA Contact: yulian → stephend
Product: MailNews → Core
Assignee: srilatha → nobody
Priority: P3 → --
QA Contact: stephend → simple-mapi
Target Milestone: mozilla1.1alpha → ---
(Assignee)

Updated

10 years ago
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.