Merge lp:~alex-weej/compiz/resize-mode into lp:~compiz/compiz/ubuntu

Proposed by Alexander Jones
Status: Rejected
Rejected by: Travis Watkins
Proposed branch: lp:~alex-weej/compiz/resize-mode
Merge into: lp:~compiz/compiz/ubuntu
Diff against target: None lines
To merge this branch: bzr merge lp:~alex-weej/compiz/resize-mode
Reviewer Review Type Date Requested Status
Travis Watkins (community) Disapprove
Review via email: mp+5880@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexander Jones (alex-weej) wrote :

Currently the default resize mode of "rectangle" is bad because of two reasons:

1. Reduced usability. When resizing e.g. a web browser, I have no idea if I've made it the "right size" until I release the mouse pointer. Resizing a window becomes a slower process of trial and error. With the normal resizing mode, although it may be quite sluggish on some drivers with bad 2D performance, at least you can see much more rapidly what the result of a resize is.

2. Bad appearance. Not only does the rectangle obscure the contents of a window, but it's some strange shade of blue that doesn't fit with any theme.

This branch removes the delta from upstream configuration and restores the normal resize mode as the default mode.

Revision history for this message
Travis Watkins (amaranth) wrote :

Normal resize mode still had issues on fglrx last time I checked so we can't change this default.

review: Disapprove

Unmerged revisions

266. By Alexander Jones

Revert to upstream settings for window resizing; i.e. actually resize the window live rather than drawing a blue rectangle

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/patches/029_default_options'
--- debian/patches/029_default_options 2009-03-17 18:02:07 +0000
+++ debian/patches/029_default_options 2009-04-24 18:01:40 +0000
@@ -118,19 +118,6 @@
118 <min>0</min>118 <min>0</min>
119 <max>4</max>119 <max>4</max>
120 <desc>120 <desc>
121Index: compiz-0.8.2/metadata/resize.xml.in
122===================================================================
123--- compiz-0.8.2.orig/metadata/resize.xml.in 2009-02-15 10:10:23.000000000 +0100
124+++ compiz-0.8.2/metadata/resize.xml.in 2009-03-17 19:00:02.000000000 +0100
125@@ -40,7 +40,7 @@
126 <option name="mode" type="int">
127 <_short>Default Resize Mode</_short>
128 <_long>Default mode used for window resizing</_long>
129- <default>0</default>
130+ <default>2</default>
131 <min>0</min>
132 <max>3</max>
133 <desc>
134Index: compiz-0.8.2/metadata/fade.xml.in121Index: compiz-0.8.2/metadata/fade.xml.in
135===================================================================122===================================================================
136--- compiz-0.8.2.orig/metadata/fade.xml.in 2009-03-17 19:00:09.000000000 +0100123--- compiz-0.8.2.orig/metadata/fade.xml.in 2009-03-17 19:00:09.000000000 +0100

Subscribers

People subscribed via source and target branches

to all changes: