Closed
Bug 126730
Opened 23 years ago
Closed 17 years ago
full screen: problem with maximize/fullscreen/restore (still broken on Linux)
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: david, Assigned: dennis.mckenzie)
References
(Blocks 1 open bug)
Details
(Keywords: platform-parity, Whiteboard: [adt3])
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
roc
:
approval1.9+
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.8+) Gecko/20020220
BuildID: 2002022003
There's a problem with browser window size when you maximize/fullscreen/restore.
Reproducible: Always
Steps to Reproduce:
1) set theme to Modern (haven't tried Classic)
2) set the window size to something other than maximized; note the size and
position of the window
3) maximize browser window
4) enter fullscreen mode
5) exit fullscreen mode
Actual Results: a) the window is at the Maximized _size_ ...BUT
b) the window is actually in the 'Restored' state.
This leads to confusion about how to get the correctly sized/positioned
'Restored' window back, since I never (thought I) changed it.
Expected Results: I believe that the 'Restore' button in fullscreen mode should
'restore' to whatever state the window was in _before_ fullscreen mode took
effect, not necessarily the Windows 'Restored' mode.
fix: No, you don't need to edit localstore.rdf :-}
Make sure the window is in the 'Restored' state and then move it so you can see
the borders and then resize it.
Comment 1•23 years ago
|
||
*** Bug 126678 has been marked as a duplicate of this bug. ***
Comment 3•23 years ago
|
||
I also see this on win98se, same build, confirming.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
Comment 4•23 years ago
|
||
Moving Netscape owned 0.9.9 and 1.0 bugs that don't have an nsbeta1, nsbeta1+,
topembed, topembed+, Mozilla0.9.9+ or Mozilla1.0+ keyword. Please send any
questions or feedback about this to adt@netscape.com. You can search for
"Moving bugs not scheduled for a project" to quickly delete this bugmail.
Target Milestone: mozilla1.0 → mozilla1.2
Comment 5•23 years ago
|
||
This bug is also present in Classic theme.
Comment 6•23 years ago
|
||
This effectively breaks Restore functionality for anyone who uses full screen
mode. Nominating for nsbeta1. Also clearing Target Milestone to trigger
reevaluation since this was targeted at 1.0 before it was auto-moved.
Keywords: nsbeta1
Target Milestone: mozilla1.2alpha → ---
Comment 7•23 years ago
|
||
resolving as invalid, current behavior is as intended.
Comment 8•23 years ago
|
||
I think you are confusing this bug with bug 126678. Reopening.
Is it really by design that if you maximize the browser window, enter full
screen, and then exit full screen, the window does *not* go back to maximized
state, rather, it goes to *restored* state but positions itself as if it was
maximized?
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 9•23 years ago
|
||
I noticed that before it goes to maximized size, it first goes to restored
position with the correct dimensions, then it resizes to make it appear maximized.
The resizing changes the default restored state, thus we must manually readjust
the browser size. I suggest that in the meantime, if this bug isn't fixed soon,
we should just remove the resizing and leave it restored.
It's easier to maximize than to manually resize to the desired restore size
everytime. It's like replacing a bug with a lesser bug. :)
Updated•23 years ago
|
Comment 10•23 years ago
|
||
No, not confusing with any other bug, just not reading carefully enough while
trying to reproduce. Renominating, but doubt we can get to this, so adding
helpwanted keyword.
Comment 11•23 years ago
|
||
nsbeta1+ per Nav triage team, ->1.0, adt3
Comment 12•23 years ago
|
||
2002033108 OS/2 800 X 600
Once I hit F11 whilst fullscreen, the only way I can get restored back is to
resize manually, both Classic and Modern. I'd change the severity to major if I
had the power, and it is definitely not windoze only, which also I cannot change.
Comment 13•23 years ago
|
||
Where's the code that resizes the window when entering/exiting fullscreen mode?
It looks like it's restoring to the proper size, but then being resized the the
max window size. If the code looks anything like I hope it does, it should be
pretty straight-forward to fix.
Comment 14•23 years ago
|
||
*** Bug 133239 has been marked as a duplicate of this bug. ***
Comment 15•23 years ago
|
||
Platform/OS to All/All based on comment 12.
Is this bug really going to make it for 1.0?
OS: Windows 98 → All
Hardware: PC → All
Comment 16•23 years ago
|
||
Another symptom of this problem, as reported in bug 133239, is that Alt-Spacebar
| Restore from fullscreen mode does not work right. Alt-Spacebar | Restore
should exit fullscreen mode and return the window to its previous position.
Comment 17•23 years ago
|
||
Andrew: you're seeing bug 127014, "restore" command (from taskbar or alt+space)
doesn't disabled in full screen mode.
Comment 19•23 years ago
|
||
*** Bug 132192 has been marked as a duplicate of this bug. ***
Comment 20•23 years ago
|
||
Mass moving nsbeta1+/adt3 bugs assigned to Navigator team engineers out to
target milestone 1.0.1. Please confine your attentions to driving down our list
of TM 1.0 bugs for beta. Better to help, debug, or test one of them than fix
one of these.
Target Milestone: mozilla1.0 → mozilla1.0.1
Comment 21•23 years ago
|
||
Changing nsbeta1+ [adt3] bugs to nsbeta1- on behalf of the adt. If you have any
questions about this, please email adt@netscape.com. You can search for
"changing adt3 bugs" to quickly find and delete these bug mails.
Keywords: nsbeta1-
Comment 22•23 years ago
|
||
Changing nsbeta1+ [adt3] bugs to nsbeta1- on behalf of the adt. If you have any
questions about this, please email adt@netscape.com. You can search for
"changing adt3 bugs" to quickly find and delete these bug mails.
Keywords: nsbeta1+
Comment 23•23 years ago
|
||
This should be pretty straight forward, right? Current behavior is
1) Restore
2) Resize
Just change this to
1) Restore
2) Maximize.
I know you eventually want to get rid of step 1 (go directly from
Full Screen to Maximized while remembering restore coordinates) but
this should be good enough for now, IMO.
Comment 24•23 years ago
|
||
*** Bug 141861 has been marked as a duplicate of this bug. ***
Updated•23 years ago
|
Keywords: mozilla1.0
Comment 25•22 years ago
|
||
*** Bug 147647 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
Keywords: mozilla1.1
Comment 27•22 years ago
|
||
Keeps the sizemode of window when turned full screen on and restores the
sizemode if it's maxzimized when turned full screen off.
Comment 28•22 years ago
|
||
Comment on attachment 92069 [details] [diff] [review]
patch for Windows
I think it might be simpler than this - the problem is that the chrome and
resize is happening in the same order both times, but one should be the reverse
of the other; my preference is to hide the OS chrome before resizing the
window.
Comment 29•22 years ago
|
||
no, this is not exactly the problem. we can descibe what we did when toggle full
screen here:
1) normal size
2) maximized window
3) turned on full screen, recorded mOriginalPos and mOriginalSize, e.g. (0, 0),
(800, 600),
4) turned off full screen, restore window size/postion by using
mOriginalPos/mOriginalSize, then the window is maximize-sized but not maximized,
because we didn't remember the original sizemode at all.
Comment 30•22 years ago
|
||
Is the patch from Comment #27 ready for a review/super review or not?
It sounds like it fixes the bug.
Another annoying behavior caused by this bug is when
middle-clicking on a link after the window was restored from fullscreen mode and
did not clicked the maximize icon , the new window that is created is shifted a
little toward the lower-right corner.
Updated•22 years ago
|
Keywords: mozilla1.1 → mozilla1.2
Comment 31•22 years ago
|
||
Yes, it's ready for r= (see there is "review" in the keywords list), but it's
still incompleted, because the patch only fixed the problem on Windows. I'll
try to fix the *nix part in a near future.
Comment 32•22 years ago
|
||
kyle, see also the patch I'm working on over on bug 176640. I don't think it's
difficult to merge the changes, but my patch puts all of the fullscreen logic in
nsBaseWidget.
Comment 33•22 years ago
|
||
Comment on attachment 92069 [details] [diff] [review]
patch for Windows
bryner, your fix for bug 176640 does fix this problem on Windows, but seems
it's still on Linux.
Attachment #92069 -
Attachment is obsolete: true
Updated•22 years ago
|
Keywords: mozilla1.0 → mozilla1.3
Comment 35•22 years ago
|
||
I don't see this problem on Redhat 7.3 (though fullscreen is not really "full
screen" as that only seems to work now under Metacity).
Comment 36•22 years ago
|
||
note that fullscreen does work for me with sawfish2 & recent trunk (gtk1)
Updated•22 years ago
|
Keywords: mozilla1.2
Updated•22 years ago
|
Flags: blocking1.4b?
Keywords: mozilla1.3 → pp
Updated•22 years ago
|
Flags: blocking1.4b? → blocking1.4b-
Comment 37•21 years ago
|
||
This bug also seems to affect the browser's saved window state after closing.
This is best seen on a dual display machine (or possibly only).
1) Open browser and place in secondary display.
2) Maximize browser window.
3) Enter Fullscreen mode.
4) Exit Fullscreen mode (Notice the window is maximized as expected).
5) Close the browser.
6) Re-open the browser.
Notice that the browser is not maximized as it should be and was before being
closed--instead it is in a restored state but taking up the entire window, sans
the taskbar. I imagine this bug is widespread.
I have tested and verified this behavior in both of the following environments:
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4) Gecko/20030624
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5b) Gecko/20030729 Mozilla
Firebird/0.6.1
Updated•21 years ago
|
Flags: blocking1.6a?
Target Milestone: mozilla1.0.1 → ---
Comment 38•21 years ago
|
||
*** Bug 220994 has been marked as a duplicate of this bug. ***
Comment 39•21 years ago
|
||
*** Bug 227315 has been marked as a duplicate of this bug. ***
Comment 40•21 years ago
|
||
We only need a fix for Linux to get this one closed.
Flags: blocking1.7a?
Summary: full screen: problem with maximize/fullscreen/restore → full screen: problem with maximize/fullscreen/restore (still broken on Linux)
Comment 41•21 years ago
|
||
Not going to hold 1.7alpha for this. We'd consider taking a reviewed patch if
someone comes up with a fix
Flags: blocking1.7a? → blocking1.7a-
Comment 42•21 years ago
|
||
after maximize (F11), the navigation bar which should be minimized become
maximize and there is another minimized navigation bar ......
Comment 43•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.10) Gecko/20050716
Firefox/1.0.6
Still happening on XP, with one thing to note: If you follow the instructions
in the original stage, the window will be maximized. When you restore the
window, the window is now larger (at least length-wise) than the maximized
window. I have a 1280 x 1024 screen resolution with a 2-line taskbar at the
bottom, leading me to believe that the "restored" window is actually 1280x1024.
Comment 44•19 years ago
|
||
*** Bug 248473 has been marked as a duplicate of this bug. ***
Comment 45•19 years ago
|
||
This is still present in XP with Firefox
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050919
Firefox/1.4 ID:2005091906
It would be really nice if this got fixed in time for 1.5 as it's a pretty
visible polish bug that has been an annoyance forever.
I'm seeing this using the following steps:
1. Window is in restored state to begin, at a smaller size than the screen
2. Maximize
3. Go into fullscreen
4. Come out of fullscreen
5. Firefox is now in maximized mode, as expected
6. If you try and go back to restore, window size and position has been lost and
it is stretched to the max screen size.
Nominated blocking1.8b5.
Flags: blocking1.8b5?
Updated•19 years ago
|
Flags: blocking1.8b5? → blocking1.8b5-
Comment 46•18 years ago
|
||
seems to wfm on winxp in ff 1.5.0.4
Updated•18 years ago
|
Assignee: bryner → jag
QA Contact: jrgmorrison → xptoolkit.widgets
Assignee | ||
Comment 47•17 years ago
|
||
Skips call to MakeFullScreenInternal, gdk_window_fullscreen and gdk_window_unfullscreen handles this on its own.
Attachment #295017 -
Flags: superreview?
Attachment #295017 -
Flags: review?
Assignee | ||
Updated•17 years ago
|
Attachment #295017 -
Flags: superreview?(roc)
Attachment #295017 -
Flags: superreview?
Attachment #295017 -
Flags: review?(roc)
Attachment #295017 -
Flags: review?
Assignee | ||
Updated•17 years ago
|
Attachment #295017 -
Flags: superreview?(roc)
Maybe if that GTK_CHECK_VERSION call fails (can it ever really?) we should call MakeFullScreenInternal(aFullScreen) after HideWindowChrome. The best way to achieve that is probably to just have the #else path call nsBaseWidget::MakeFullScreen. And if we're doing *that*, we should remove nsBaseWidget::MakeFullScreenInternal and just put it inline into nsBaseWidget::MakeFullScreen.
Assignee | ||
Comment 49•17 years ago
|
||
Restores call to nsBaseWidget::MakeFullScreen for old gtk versions.
MakeFullScreenInternal integrated into MakeFullScreen.
Removed duplicate call to HideWindowChrome.
Attachment #295296 -
Flags: review?(roc)
Comment on attachment 295296 [details] [diff] [review]
fixes the maximize/fullscreen/restore cycle on newer gtk
cool. This a simple fix, we should land it for FF3. Thanks!!!
Attachment #295296 -
Flags: superreview+
Attachment #295296 -
Flags: review?(roc)
Attachment #295296 -
Flags: review+
Attachment #295296 -
Flags: approval1.9+
Keywords: checkin-needed
Updated•17 years ago
|
Assignee: jag → dennis.mckenzie
Updated•17 years ago
|
Attachment #295017 -
Attachment is obsolete: true
Attachment #295017 -
Flags: review?(roc)
Comment 51•17 years ago
|
||
Checking in widget/src/gtk2/nsWindow.cpp;
/cvsroot/mozilla/widget/src/gtk2/nsWindow.cpp,v <-- nsWindow.cpp
new revision: 1.247; previous revision: 1.246
done
Checking in widget/src/xpwidgets/nsBaseWidget.cpp;
/cvsroot/mozilla/widget/src/xpwidgets/nsBaseWidget.cpp,v <-- nsBaseWidget.cpp
new revision: 1.168; previous revision: 1.167
done
Checking in widget/src/xpwidgets/nsBaseWidget.h;
/cvsroot/mozilla/widget/src/xpwidgets/nsBaseWidget.h,v <-- nsBaseWidget.h
new revision: 1.93; previous revision: 1.92
done
Status: NEW → RESOLVED
Closed: 23 years ago → 17 years ago
Keywords: checkin-needed,
helpwanted
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M11
See Also: → https://launchpad.net/bugs/30522
You need to log in
before you can comment on or make changes to this bug.
Description
•