Hi,

currently, all sorting order of the core and core plugin options inside
the metadata files is based on the order they are listed in the C source
files. Unfortunately, that's not optimal for automatic settings manager
GUI generation.

DO you think it's ok to sort them by functional groups? This would have
the advantage of settings managers being aware of the functional
grouping, which would help a lot in creating them.

I have attached a patch which re-sorts the core options a bit to reflect
that - what do you think about it? Is this change ok to go in?

Regards,

Danny
diff --git a/metadata/core.xml.in b/metadata/core.xml.in
index 8424ab1..7c67419 100644
--- a/metadata/core.xml.in
+++ b/metadata/core.xml.in
@@ -32,6 +32,11 @@
 		<_long>Click on window moves input focus to it</_long>
 		<default>true</default>
 	    </option>
+	    <option name="raise_on_click" type="bool">
+		<_short>Raise On Click</_short>
+		<_long>Raise windows when clicked</_long>
+		<default>true</default>
+	    </option>
 	    <option name="autoraise" type="bool">
 		<_short>Auto-Raise</_short>
 		<_long>Raise selected windows after interval</_long>
@@ -44,6 +49,16 @@
 		<min>0</min>
 		<max>10000</max>
 	    </option>
+	    <option name="unredirect_fullscreen_windows" type="bool">
+		<_short>Unredirect Fullscreen Windows</_short>
+		<_long>Allow drawing of fullscreen windows to not be redirected to offscreen pixmaps</_long>
+		<default>false</default>
+	    </option>
+	    <option name="audible_bell" type="bool">
+		<_short>Audible Bell</_short>
+		<_long>Audible system beep</_long>
+		<default>true</default>
+	    </option>
 	    <option name="close_window" type="action">
 		<_short>Close Window</_short>
 		<_long>Close active window</_long>
@@ -305,16 +320,6 @@
 		    <key>&lt;Control&gt;&lt;Alt&gt;d</key>
 		</default>
 	    </option>
-	    <option name="raise_on_click" type="bool">
-		<_short>Raise On Click</_short>
-		<_long>Raise windows when clicked</_long>
-		<default>true</default>
-	    </option>
-	    <option name="audible_bell" type="bool">
-		<_short>Audible Bell</_short>
-		<_long>Audible system beep</_long>
-		<default>true</default>
-	    </option>
 	    <option name="toggle_window_maximized" type="action">
 		<_short>Toggle Window Maximized</_short>
 		<_long>Toggle active window maximized</_long>
@@ -367,16 +372,21 @@
 	    </option>
 	</display>
 	<screen>
-	    <option name="detect_refresh_rate" type="bool">
-		<_short>Detect Refresh Rate</_short>
-		<_long>Automatic detection of refresh rate</_long>
-		<default>true</default>
-	    </option>
 	    <option name="lighting" type="bool">
 		<_short>Lighting</_short>
 		<_long>Use diffuse light when screen is transformed</_long>
 		<default>true</default>
 	    </option>
+	    <option name="sync_to_vblank" type="bool">
+		<_short>Sync To VBlank</_short>
+		<_long>Only perform screen updates during vertical blanking period</_long>
+		<default>true</default>
+	    </option>
+	    <option name="detect_refresh_rate" type="bool">
+		<_short>Detect Refresh Rate</_short>
+		<_long>Automatic detection of refresh rate</_long>
+		<default>true</default>
+	    </option>
 	    <option name="refresh_rate" type="int">
 		<_short>Refresh Rate</_short>
 		<_long>The rate at which the screen is redrawn (times/second)</_long>
@@ -398,6 +408,13 @@
 		<min>1</min>
 		<max>32</max>
 	    </option>
+	    <option name="number_of_desktops" type="int">
+		<_short>Number of Desktops</_short>
+		<_long>Number of virtual desktops</_long>
+		<default>1</default>
+		<min>1</min>
+		<max>36</max>
+	    </option>
 	    <option name="opacity_step" type="int">
 		<_short>Opacity Step</_short>
 		<_long>Opacity change step</_long>
@@ -405,28 +422,11 @@
 		<min>1</min>
 		<max>50</max>
 	    </option>
-	    <option name="unredirect_fullscreen_windows" type="bool">
-		<_short>Unredirect Fullscreen Windows</_short>
-		<_long>Allow drawing of fullscreen windows to not be redirected to offscreen pixmaps</_long>
-		<default>false</default>
-	    </option>
 	    <option name="default_icon" type="string">
 		<_short>Default Icon</_short>
 		<_long>Default window icon image</_long>
 		<default>icon</default>
 	    </option>
-	    <option name="sync_to_vblank" type="bool">
-		<_short>Sync To VBlank</_short>
-		<_long>Only perform screen updates during vertical blanking period</_long>
-		<default>true</default>
-	    </option>
-	    <option name="number_of_desktops" type="int">
-		<_short>Number of Desktops</_short>
-		<_long>Number of virtual desktops</_long>
-		<default>1</default>
-		<min>1</min>
-		<max>36</max>
-	    </option>
 	    <option name="detect_outputs" type="bool">
 		<_short>Detect Outputs</_short>
 		<_long>Automatic detection of output devices</_long>
_______________________________________________
compiz mailing list
compiz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/compiz

Reply via email to