GNOME Bugzilla – Bug 94815
Window moves when resizing
Last modified: 2004-12-22 21:47:04 UTC
Package: metacity Severity: normal Version: 2.4.1 Synopsis: Window moves when resizing Bugzilla-Product: metacity Bugzilla-Component: general Description: Description of Problem: If I open up galeon and resize it so that the right side is flush against the ride side of the monitor. And then I drag the left side to resize and I resize so that its shrunken to zero width and then I make it big again the right side of the window is moved away from the right side of the monitor. Steps to reproduce the problem: 1. Open up galeon (or a window with no minimum width) 2. Move it so the right side of the window is against the ride side of the monitor. 3. Drag the left side of the window until the window has zero width and then drag it back. Actual Results: The window has moved away from the right side of the monitor. Expected Results: Window shrinks and then grows. How often does this happen? Everytime. Additional Information: ------- Bug moved to this database by unknown@bugzilla.gnome.org 2002-10-03 21:14 ------- Reassigning to the default owner of the component, hp@redhat.com.
The new constraints rewrite in constraints.c is intended to address this issue among others.
Batch adding GNOME2 keyword to Metacity bugs. Sorry for the spam.
This bug still exists even with the constraints rewrite. It is highly funky.
Created attachment 14959 [details] [review] this patch fixes this problem by making the onscreen pixel constraint scale down with window size
Created attachment 14960 [details] [review] updated patch; old one could allow undecorated 1x1 windows completely off screen.
marking PATCH/high
Does this look OK to you Havoc? To me, this seems like the Right Fix (TM). Basically, if a window is smaller than TITLEBAR_LENGTH_ONSCREEN, we shouldn't force it to be onscreen by a width larger than the window size. That's what was causing the window to move when resizing it -- the onscreen constraint interacting with the user resize request. -Rob
This patch does look good, just add space in "MIN(" ;-) Please commit, thanks.
committed. And the MIN's converted to your evil bastardized convention. You're such a taskmaster :-)