QuickLaunch - app startup pref ignored

VERIFIED FIXED

Status

Core Graveyard
QuickLaunch (AKA turbo mode)
--
minor
VERIFIED FIXED
10 years ago
5 years ago

People

(Reporter: Chris Thomas (CTho) [formerly cst@andrew.cmu.edu cst@yecc.com], Assigned: neil@parkwaycc.co.uk)

Tracking

({fixed-seamonkey1.1.1})

1.8 Branch
x86
Windows XP
fixed-seamonkey1.1.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

573 bytes, patch
Chris Thomas (CTho) [formerly cst@andrew.cmu.edu cst@yecc.com]
: review+
Biesinger
: superreview+
Details | Diff | Splinter Review
In SM1.1., with QuickLaunch running, no matter what boxes are checked under Appearance->"When SeaMonkey starts up, open", a shortcut that runs seamonkey.exe with no arguments results in a browser only.

Updated

10 years ago
Assignee: general → quicklaunch
Component: General → QuickLaunch (AKA turbo mode)
Product: Mozilla Application Suite → Core
QA Contact: general
(Assignee)

Comment 1

10 years ago
Created attachment 255587 [details] [diff] [review]
Proposed patch

OK, so the issue here is that seamonkey -turbo needs to inhibit the default preferences. It does this by setting an mShouldShowUI flag to false, but this flag never gets set to true again. This wasn't a problem in 1.0.x because bug 58523 changed the order of startup tests and this code only used to run once.
Assignee: quicklaunch → neil
Status: NEW → ASSIGNED
Attachment #255587 - Flags: review?
Duplicate of this bug: 370798
(Assignee)

Comment 3

10 years ago
Comment on attachment 255587 [details] [diff] [review]
Proposed patch

See previous comment...
Attachment #255587 - Flags: superreview?(jag)
Attachment #255587 - Flags: review?(cst)
Attachment #255587 - Flags: review?
Comment on attachment 255587 [details] [diff] [review]
Proposed patch

I think this fixed bug 368421 too.
Attachment #255587 - Flags: review?(cst) → review+
suspect there is _at least_ one, much older dupe. sorry, i read quite of few but don't have number(s)
Comment on attachment 255587 [details] [diff] [review]
Proposed patch

could you diff with more context and -p in the future?
Attachment #255587 - Flags: superreview?(jag) → superreview+

Updated

10 years ago
Keywords: fixed-seamonkey1.1.1
Duplicate of this bug: 368421
Duplicate of this bug: 367680
Duplicate of this bug: 366569
Duplicate of this bug: 367756
Any reason this bug isn't marked as resolved?
(Assignee)

Comment 12

10 years ago
(In reply to comment #11)
>Any reason this bug isn't marked as resolved?
Because I was checking with KaiRo as to whether it would make 1.1.1 and overlooked that he'd only set the 1.1.1 flag and not resolved it too.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Verifying based on user feedback.
Status: RESOLVED → VERIFIED
Component: QuickLaunch (AKA turbo mode) → QuickLaunch (AKA turbo mode)
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.