[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [20312] branches/blender2.5/blender: * removed some arbitrary range limits on property values

Matt Ebb matt at mke3.net
Thu May 21 09:40:43 CEST 2009


Revision: 20312
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=20312
Author:   broken
Date:     2009-05-21 09:40:43 +0200 (Thu, 21 May 2009)

Log Message:
-----------
* removed some arbitrary range limits on property values
* turned some slider UI controls back into number fields

Note: Sliders should only be used when representing a property that's 
conceptually a percentage or ratio. For arbitrary numbers, use a number field.

For example, a lamp's energy can theoretically be anything - 0.1, 3.5, 125.0, it 
doesn't matter - it's just a number, not a percentage. So this should be a 
number field.

Modified Paths:
--------------
    branches/blender2.5/blender/release/ui/buttons_data_curve.py
    branches/blender2.5/blender/release/ui/buttons_data_lamp.py
    branches/blender2.5/blender/release/ui/buttons_material.py
    branches/blender2.5/blender/source/blender/makesrna/intern/rna_curve.c
    branches/blender2.5/blender/source/blender/makesrna/intern/rna_lamp.c

Modified: branches/blender2.5/blender/release/ui/buttons_data_curve.py
===================================================================
--- branches/blender2.5/blender/release/ui/buttons_data_curve.py	2009-05-21 07:28:02 UTC (rev 20311)
+++ branches/blender2.5/blender/release/ui/buttons_data_curve.py	2009-05-21 07:40:43 UTC (rev 20312)
@@ -69,7 +69,7 @@
 			sub.itemR(curve, "taper_object")
 			sub = split.column()
 			sub.itemL(text="Bevel:")
-			sub.itemR(curve, "bevel_depth", text="Depth", slider=True)
+			sub.itemR(curve, "bevel_depth", text="Depth")
 			sub.itemR(curve, "bevel_resolution", text="Resolution")
 			sub.itemR(curve, "bevel_object")
 

Modified: branches/blender2.5/blender/release/ui/buttons_data_lamp.py
===================================================================
--- branches/blender2.5/blender/release/ui/buttons_data_lamp.py	2009-05-21 07:28:02 UTC (rev 20311)
+++ branches/blender2.5/blender/release/ui/buttons_data_lamp.py	2009-05-21 07:40:43 UTC (rev 20312)
@@ -28,7 +28,7 @@
 		
 		sub = split.column()
 		sub.itemR(lamp, "color")
-		sub.itemR(lamp, "energy", slider=True)
+		sub.itemR(lamp, "energy")
 		sub.itemR(lamp, "distance")
 		sub.itemR(lamp, "negative")
 	

Modified: branches/blender2.5/blender/release/ui/buttons_material.py
===================================================================
--- branches/blender2.5/blender/release/ui/buttons_material.py	2009-05-21 07:28:02 UTC (rev 20311)
+++ branches/blender2.5/blender/release/ui/buttons_material.py	2009-05-21 07:40:43 UTC (rev 20312)
@@ -50,7 +50,7 @@
 		flow = layout.column_flow()
 		flow.itemR(sss, "error_tolerance")
 		flow.itemR(sss, "ior")
-		flow.itemR(sss, "scale", slider=True)
+		flow.itemR(sss, "scale")
 		
 		row = layout.row()
 		row.column().itemR(sss, "color")
@@ -59,8 +59,8 @@
 		flow = layout.column_flow()
 		flow.itemR(sss, "color_factor", slider=True)
 		flow.itemR(sss, "texture_factor", slider=True)
-		flow.itemR(sss, "front", slider=True)
-		flow.itemR(sss, "back", slider=True)
+		flow.itemR(sss, "front")
+		flow.itemR(sss, "back")
 		
 class MATERIAL_PT_raymir(MaterialButtonsPanel):
 	__idname__= "MATERIAL_PT_raymir"
@@ -84,7 +84,7 @@
 		
 		sub = split.column()
 		sub.itemR(raym, "reflect", text="RayMir", slider=True)
-		sub.itemR(raym, "fresnel", slider=True)
+		sub.itemR(raym, "fresnel")
 		sub.itemR(raym, "fresnel_fac", text="Fac", slider=True)
 		
 		sub = split.column()
@@ -120,7 +120,7 @@
 		
 		sub = split.column()
 		sub.itemR(rayt, "ior")
-		sub.itemR(rayt, "fresnel", slider=True)
+		sub.itemR(rayt, "fresnel")
 		sub.itemR(rayt, "fresnel_fac", text="Fac", slider=True)
 		
 		sub = split.column()
@@ -131,7 +131,7 @@
 		flow = layout.column_flow()
 		flow.itemR(rayt, "filter", slider=True)
 		flow.itemR(rayt, "limit")
-		flow.itemR(rayt, "falloff", slider=True)
+		flow.itemR(rayt, "falloff")
 		flow.itemR(rayt, "specular_opacity", slider=True)
 		flow.itemR(rayt, "depth")
 		
@@ -152,8 +152,8 @@
 		
 		sub = split.column()
 		sub.itemL(text="General Settings:")
-		sub.itemR(halo, "size", slider=True)
-		sub.itemR(halo, "hardness", slider=True)
+		sub.itemR(halo, "size")
+		sub.itemR(halo, "hardness")
 		sub.itemR(halo, "add", slider=True)
 		
 		sub = split.column()
@@ -175,20 +175,20 @@
 	
 		sub = row.column()
 		if (halo.ring):
-			sub.itemR(halo, "rings", slider=True)
+			sub.itemR(halo, "rings")
 		if (halo.lines):
-			sub.itemR(halo, "line_number", slider=True)
+			sub.itemR(halo, "line_number")
 		if (halo.ring or halo.lines):
 			sub.itemR(halo, "seed")
 		if (halo.star):
-			sub.itemR(halo, "star_tips", slider=True)
+			sub.itemR(halo, "star_tips")
 		if (halo.flare_mode):
 			sub.itemL(text="Flare:")
-			sub.itemR(halo, "flare_size", text="Size", slider=True)
-			sub.itemR(halo, "flare_subsize", text="Subsize", slider=True)
-			sub.itemR(halo, "flare_boost", text="Boost", slider=True)
+			sub.itemR(halo, "flare_size", text="Size")
+			sub.itemR(halo, "flare_subsize", text="Subsize")
+			sub.itemR(halo, "flare_boost", text="Boost")
 			sub.itemR(halo, "flare_seed", text="Seed")
-			sub.itemR(halo, "flares_sub", text="Sub", slider=True)
+			sub.itemR(halo, "flares_sub", text="Sub")
 				
 bpy.types.register(MATERIAL_PT_material)
 bpy.types.register(MATERIAL_PT_raymir)

Modified: branches/blender2.5/blender/source/blender/makesrna/intern/rna_curve.c
===================================================================
--- branches/blender2.5/blender/source/blender/makesrna/intern/rna_curve.c	2009-05-21 07:28:02 UTC (rev 20311)
+++ branches/blender2.5/blender/source/blender/makesrna/intern/rna_curve.c	2009-05-21 07:40:43 UTC (rev 20312)
@@ -479,41 +479,42 @@
 	prop= RNA_def_property(srna, "bevel_resolution", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "bevresol");
 	RNA_def_property_range(prop, 0, 32);
+	RNA_def_property_ui_range(prop, 0, 32, 1.0, 0);
 	RNA_def_property_ui_text(prop, "Bevel Resolution", "Bevel resolution when depth is non-zero and no specific bevel object has been defined.");
 	
 	prop= RNA_def_property(srna, "width", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "width");
-	RNA_def_property_range(prop, 0.0f, 2.0f);
+	RNA_def_property_ui_range(prop, 0, 2.0, 0.1, 0);
 	RNA_def_property_ui_text(prop, "Width", "Scale the original width (1.0) based on given factor.");
 	
 	prop= RNA_def_property(srna, "extrude", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "ext1");
-	RNA_def_property_range(prop, 0.0f, 100.0f);
+	RNA_def_property_ui_range(prop, 0, 100.0, 0.1, 0);
 	RNA_def_property_ui_text(prop, "Extrude", "Amount of curve extrusion when not using a bevel object.");
 	
 	prop= RNA_def_property(srna, "bevel_depth", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "ext2");
-	RNA_def_property_range(prop, 0.0f, 2.0f);
+	RNA_def_property_ui_range(prop, 0, 100.0, 0.1, 0);
 	RNA_def_property_ui_text(prop, "Bevel Depth", "Bevel depth when not using a bevel object.");
 	
 	prop= RNA_def_property(srna, "resolution_u", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "resolu");
-	RNA_def_property_range(prop, 1, 1024);
+	RNA_def_property_ui_range(prop, 1, 1024, 1, 0);
 	RNA_def_property_ui_text(prop, "Resolution U", "Surface resolution in U direction.");
 	
 	prop= RNA_def_property(srna, "resolution_v", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "resolv");
-	RNA_def_property_range(prop, 1, 1024);
+	RNA_def_property_ui_range(prop, 1, 1024, 1, 0);
 	RNA_def_property_ui_text(prop, "Resolution V", "Surface resolution in V direction.");
 	
 	prop= RNA_def_property(srna, "render_resolution_u", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "resolu_ren");
-	RNA_def_property_range(prop, 0, 1024);
+	RNA_def_property_ui_range(prop, 1, 1024, 1, 0);
 	RNA_def_property_ui_text(prop, "Render Resolution U", "Surface resolution in U direction used while rendering. Zero skips this property.");
 	
 	prop= RNA_def_property(srna, "render_resolution_v", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "resolv_ren");
-	RNA_def_property_range(prop, 0, 1024);
+	RNA_def_property_ui_range(prop, 1, 1024, 1, 0);
 	RNA_def_property_ui_text(prop, "Render Resolution V", "Surface resolution in V direction used while rendering. Zero skips this property.");
 	
 	/* pointers */

Modified: branches/blender2.5/blender/source/blender/makesrna/intern/rna_lamp.c
===================================================================
--- branches/blender2.5/blender/source/blender/makesrna/intern/rna_lamp.c	2009-05-21 07:28:02 UTC (rev 20311)
+++ branches/blender2.5/blender/source/blender/makesrna/intern/rna_lamp.c	2009-05-21 07:40:43 UTC (rev 20312)
@@ -270,12 +270,12 @@
 
 	prop= RNA_def_property(srna, "distance", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "dist");
-	RNA_def_property_range(prop, 0.01f, 5000.0f);
+	RNA_def_property_ui_range(prop, 0, 1000, 1.0, 2);
 	RNA_def_property_ui_text(prop, "Distance", "Falloff distance - the light is at half the original intensity at this point.");
 	RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING_DRAW, NULL);
 
 	prop= RNA_def_property(srna, "energy", PROP_FLOAT, PROP_NONE);
-	RNA_def_property_range(prop, 0.0f, 10.0f);
+	RNA_def_property_ui_range(prop, 0, 10.0, 0.1, 2);
 	RNA_def_property_ui_text(prop, "Energy", "Amount of light that the lamp emits.");
 	RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING, NULL);
 
@@ -408,14 +408,14 @@
 
 	prop= RNA_def_property(srna, (area)? "shadow_ray_samples_x": "shadow_ray_samples", PROP_INT, PROP_NONE);
 	RNA_def_property_int_sdna(prop, NULL, "ray_samp");
-	RNA_def_property_range(prop, 1, 16);
+	RNA_def_property_range(prop, 1, 64);
 	RNA_def_property_ui_text(prop, (area)? "Shadow Ray Samples": "Shadow Ray Samples X","Amount of samples taken extra (samples x samples).");
 	RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING, NULL);
 
 	if(area) {
 		prop= RNA_def_property(srna, "shadow_ray_samples_y", PROP_INT, PROP_NONE);
 		RNA_def_property_int_sdna(prop, NULL, "ray_sampy");
-		RNA_def_property_range(prop, 1, 16);
+		RNA_def_property_range(prop, 1, 64);
 		RNA_def_property_ui_text(prop, "Shadow Ray Samples Y", "Amount of samples taken extra (samples x samples).");
 		RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING, NULL);
 	}
@@ -428,7 +428,7 @@
 
 	prop= RNA_def_property(srna, "shadow_soft_size", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "soft");
-	RNA_def_property_range(prop, 0.0f, 100.0f);
+	RNA_def_property_ui_range(prop, 0, 100, 0.1, 3);
 	RNA_def_property_ui_text(prop, "Shadow Soft Size", "Light size for ray shadow sampling (Raytraced shadows).");
 	RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING, NULL);
 
@@ -489,19 +489,19 @@
 
 	prop= RNA_def_property(srna, "size", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "area_size");
-	RNA_def_property_range(prop, 0.0f, 100.0f);
+	RNA_def_property_ui_range(prop, 0, 100, 0.1, 3);
 	RNA_def_property_ui_text(prop, "Size", "Size of the area of the area Lamp, X direction size for Rectangle shapes.");
 	RNA_def_property_update(prop, NC_LAMP|ND_LIGHTING_DRAW, NULL);
 
 	prop= RNA_def_property(srna, "size_y", PROP_FLOAT, PROP_NONE);
 	RNA_def_property_float_sdna(prop, NULL, "area_sizey");
-	RNA_def_property_range(prop, 0.0f, 100.0f);
+	RNA_def_property_ui_range(prop, 0, 100, 0.1, 3);

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list