Closed Bug 334903 Opened 18 years ago Closed 18 years ago

Seamonkey update page appears in window with no navigation or tabs

Categories

(SeaMonkey :: UI Design, defect)

1.8 Branch
x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aaronclawrence, Assigned: ajschult784)

References

Details

(Keywords: fixed-seamonkey1.1a, Whiteboard: fixed-seamonkey1.0.3)

Attachments

(1 file)

If you click Yes to update, when the "update available" appears, then you get a new window with no navigation (back, forward, URL etc) and no tabs bar. This makes it quite difficult to use. 
Expected: a normal browser window; actually, a normal browser tab, because I have it set to open new windows in tabs.

Seamonkey 1.0. Have installed 1.0.1 but can't recheck on demand - there's no way to do that, is there?
http://lxr.mozilla.org/seamonkey/source/xpfe/components/updates/src/nsUpdateNotifier.js#266

I can't get update to kick in, but IIRC, windowWatcher uses its psychic powers to determine that we want a dialog because the last argument passed in is null (rather than an empty string).
Assignee: general → jag
Status: UNCONFIRMED → NEW
Component: General → XP Apps
Ever confirmed: true
QA Contact: general
(In reply to comment #0)
> 
> Seamonkey 1.0. Have installed 1.0.1 but can't recheck on demand - there's no
> way to do that, is there?
> 


If you enter in about:config, reset the preference:

update_notifications.provider.0.last_checked

to 0, restart SeaMonkey and browse the web for a while, it shouldn't pass too much time before you get the update notification.

BTW, I can confirm this behaviour in Windows (XP), and I can confirm that this doesn't happen in Linux (using IceWM).
I forgot to say that Bug 334339 is a dup of this one (or this one is a dup of 334339).
Attached patch patchSplinter Review
Let's not pretend a webpage is chrome!
Assignee: jag → ajschult
Status: NEW → ASSIGNED
Attachment #228020 - Flags: superreview?(neil)
Attachment #228020 - Flags: review?(neil)
*** Bug 334339 has been marked as a duplicate of this bug. ***
Comment on attachment 228020 [details] [diff] [review]
patch

>+        var argstring = Components.classes["@mozilla.org/supports-string;1"].
>+           createInstance(Components.interfaces.nsISupportsString);
Nit: bogus 3-space indent, and . should be at the start of the second line.
Attachment #228020 - Flags: superreview?(neil)
Attachment #228020 - Flags: superreview+
Attachment #228020 - Flags: review?(neil)
Attachment #228020 - Flags: review+
Comment on attachment 228020 [details] [diff] [review]
patch

This is on trunk, but won't actually be tested (by the masses) until it's in 1.0.x
Attachment #228020 - Flags: approval-seamonkey1.1a?
Attachment #228020 - Flags: approval-seamonkey1.0.3?
Comment on attachment 228020 [details] [diff] [review]
patch

a=me for 1.1, first-a=me for 1.0.x, still one needed to go
Attachment #228020 - Flags: approval-seamonkey1.1a? → approval-seamonkey1.1a+
Comment on attachment 228020 [details] [diff] [review]
patch

a=me for 1.0.3
Attachment #228020 - Flags: approval-seamonkey1.0.3? → approval-seamonkey1.0.3+
landed on both branches
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: fixed-seamonkey1.0.3
You need to log in before you can comment on or make changes to this bug.