Here's a couple quick patches to resolve some minor issues related to multiple outputs that I bumped into today. nodetectfix.patch: Enables reading outputs config (instead of 640x480 default) when using "--no-detection" switch but config says detect. constrainyoutput.patch: Allows full vertical workarea to be used in the case of multiple vertical outputs. Currently it won't allow moving a window above the current output. Also, I think I almost have a solution ready for handling the "inner" struts issue... I'm sure someone out there besides myself cares. ;) Cheers! ...MC -------------- next part -------------- A non-text attachment was scrubbed... Name: nodetectfix.patch Type: application/octet-stream Size: 400 bytes Desc: not available Url : http://lists.freedesktop.org/archives/compiz/attachments/20061213/c218ac70/nodetectfix.obj -------------- next part -------------- A non-text attachment was scrubbed... Name: constrainyoutput.patch Type: application/octet-stream Size: 799 bytes Desc: not available Url : http://lists.freedesktop.org/archives/compiz/attachments/20061213/c218ac70/constrainyoutput.obj
David Reveman
2006-Dec-18 07:26 UTC
[compiz] Quick patches for no-detection and constrain y
On Wed, 2006-12-13 at 22:32 -0700, Mike Cook wrote:> Here's a couple quick patches to resolve some minor issues > related to multiple outputs that I bumped into today. > > nodetectfix.patch: > Enables reading outputs config (instead of 640x480 default) > when using "--no-detection" switch but config says detect.thanks, in head now.> > constrainyoutput.patch: > Allows full vertical workarea to be used in the case of > multiple vertical outputs. Currently it won't allow moving a > window above the current output.What I like to see in the move plugin is that when a move is initiated an exact region is created which reflects where the window titlebar is allowed to be positioned. Move plugin should go through all windows struts to create this region and a simple RECT_IN_REGION check with the titlebar rect will then be used to validate a new window position.> > Also, I think I almost have a solution ready for handling the > "inner" struts issue... I'm sure someone out there besides > myself cares. ;) Cheers!Great, I definitely care. Thanks, -David