Closed
Bug 204248
Opened 21 years ago
Closed 21 years ago
No maximize/minimize buttons in new Mailnews window, and raise & lower broken
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: Matti, Assigned: emaijala+moz)
References
Details
(Keywords: regression)
Attachments
(1 file)
866 bytes,
patch
|
jag+mozilla
:
review+
jag+mozilla
:
superreview+
asa
:
approval1.4+
|
Details | Diff | Splinter Review |
win2k build 20030502.. a) open Mailnews b) select "open in new Mail Window" in the folder context menu (right click a folder) or click on a "Click here to remove all expired articles" link in an expired Newsgroup Post c) see the new opened mailnews window without minimize/maximize button but you can still resize the window
Comment 1•21 years ago
|
||
Confirmed: Linux/x86 20030503. The problem is actually much worse than it appears at first glance. In this second mozilla messenger window (using the branding names :), keyboard shortcuts for minimise/maximise don't work either. Worse, the window cannot be raised or lowered. Keyboard shortcuts for this are ignored, and window manager "raise-on-focus" doesn't happen. This makes the app suite practically unusable for anyone running 2 mozilla messenger windows, if they're big enough overlap. This all on metacity 2.4.34. The regression ocurred between 2003043011 & 2003050105 (checkout hour times in GMT).
Severity: normal → major
Flags: blocking1.4b?
Updated•21 years ago
|
OS: Windows 2000 → All
Hardware: PC → All
Updated•21 years ago
|
Summary: No maximize/minimize buttons in new Mailnews window → No maximize/minimize buttons in new Mailnews window, and raise & lower broken
Comment 2•21 years ago
|
||
Note I'm using GTK2. I wonder if this is related to the fix for: 174471 [gtk2] need to port over the nsITheme interface code to gtk2 which seems to have landed at about the right time.
I *can't* confirm this. Todays cvs (20030503) build is working fine for me with gtk2 toolkit. However i am running windowmaker windowmanager. Calum's problem is probably metacity windowmanager as stacking is a function of the windowmanager.
Comment 4•21 years ago
|
||
Interesting. I'll bet mail/news is opening the new window as a dialog. I can reproduce this here. The stacking order is the big clue.
Blocks: gtk2
Comment 5•21 years ago
|
||
It's more likely that bug 157438 caused this. Is this happening on windows, by any chance?
Reporter | ||
Comment 6•21 years ago
|
||
this happens on win2k (see my initial comment) and biesi saw it in an non gtk2 Linux build.
Updated•21 years ago
|
Flags: blocking1.4b?
Flags: blocking1.4b-
Flags: blocking1.4?
Comment 7•21 years ago
|
||
So this is or isn't a problem in the mozilla.org default builds (not GTK2)?
Comment 8•21 years ago
|
||
I've not tried a mozilla.org default build, but it is a problem in non-GTK2 builds. We build here from cvs, and I'm seeing the problem with a non-GTK2 build of 20030506. I will try a mozilla.org build.
Comment 9•21 years ago
|
||
Yup, the problem is present in mozilla.org builds too. I've just downloaded the nightly sparc 2.7 build: http://ftp.mozilla.org/pub/mozilla/nightly/latest/mozilla-sparc-sun-solaris2.7.tar.gz which is 2003050601, and the problem is still present.
Assignee | ||
Comment 11•21 years ago
|
||
I broke this when fixing bug 137661. We need to explicitly say this isn't a dialog when there are arguments.
Assignee | ||
Comment 12•21 years ago
|
||
Comment on attachment 122691 [details] [diff] [review] Patch to fix the params to OpenWindow() Jag, r/sr for this follow-up for bug 137661?
Attachment #122691 -
Flags: superreview?(jaggernaut)
Comment 13•21 years ago
|
||
Comment on attachment 122691 [details] [diff] [review] Patch to fix the params to OpenWindow() r+sr=jag
Attachment #122691 -
Flags: superreview?(jaggernaut)
Attachment #122691 -
Flags: superreview+
Attachment #122691 -
Flags: review+
Reporter | ||
Comment 14•21 years ago
|
||
*** Bug 204764 has been marked as a duplicate of this bug. ***
Updated•21 years ago
|
Attachment #122691 -
Flags: approval1.4?
Comment 15•21 years ago
|
||
Comment on attachment 122691 [details] [diff] [review] Patch to fix the params to OpenWindow() a=asa (on behalf of drivers) for checkin to 1.4.
Attachment #122691 -
Flags: approval1.4? → approval1.4+
Assignee | ||
Comment 16•21 years ago
|
||
Fix checked in to trunk.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•21 years ago
|
Flags: blocking1.4?
Comment 17•21 years ago
|
||
*** Bug 204995 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 18•21 years ago
|
||
*** Bug 205349 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 19•21 years ago
|
||
*** Bug 205414 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 20•21 years ago
|
||
*** Bug 205398 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 21•21 years ago
|
||
*** Bug 205500 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 22•21 years ago
|
||
*** Bug 205693 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 23•21 years ago
|
||
*** Bug 205722 has been marked as a duplicate of this bug. ***
Comment 24•21 years ago
|
||
*** Bug 206423 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•