Fix T59151 Bevel modifier has bad input width for percent.
authorHoward Trickey <howard.trickey@gmail.com>
Mon, 7 Jan 2019 12:29:54 +0000 (07:29 -0500)
committerHoward Trickey <howard.trickey@gmail.com>
Mon, 7 Jan 2019 12:29:54 +0000 (07:29 -0500)
The units scaling was inappropriate when the bevel value was
to be interpreted as a percent, so added a separate rna property
for "Width Percent" and made UI show the width appropriate for
current offset_type.

release/datafiles/locale
release/scripts/addons
release/scripts/startup/bl_ui/properties_data_modifier.py
source/blender/makesrna/intern/rna_modifier.c
source/blender/modifiers/intern/MOD_bevel.c

index 345b7d27872d87ae5074a83d8b9995b37bc56f1b..8eafc437295b0edc990db231fe957e2ad42af70d 160000 (submodule)
@@ -1 +1 @@
-Subproject commit 345b7d27872d87ae5074a83d8b9995b37bc56f1b
+Subproject commit 8eafc437295b0edc990db231fe957e2ad42af70d
index 40340832e3992f46034b470aef1af6f7a3a4410f..d31844cc0b3be397b5032cb0c5d8daea39584783 160000 (submodule)
@@ -1 +1 @@
-Subproject commit 40340832e3992f46034b470aef1af6f7a3a4410f
+Subproject commit d31844cc0b3be397b5032cb0c5d8daea39584783
index 52af0512a0c7881963579ff816c84077e664d9e3..a6bab561e29bf4ca8c24a3c6291a420647c01570 100644 (file)
@@ -136,7 +136,10 @@ class DATA_PT_modifiers(ModifierButtonsPanel, Panel):
         split = layout.split()
 
         col = split.column()
-        col.prop(md, "width")
+        if md.offset_type == 'PERCENT':
+            col.prop(md, "width_pct")
+        else:
+            col.prop(md, "width")
         col.prop(md, "segments")
         col.prop(md, "profile")
         col.prop(md, "material")
index 5bc925ea9695ef781335b6290e2c602754764cc8..2ef4773de207b029ce3d7df1cd4eeddd264a812c 100644 (file)
@@ -3066,7 +3066,14 @@ static void rna_def_modifier_bevel(BlenderRNA *brna)
        RNA_def_property_float_sdna(prop, NULL, "value");
        RNA_def_property_range(prop, 0, FLT_MAX);
        RNA_def_property_ui_range(prop, 0.0f, 100.0f, 0.1, 4);
-       RNA_def_property_ui_text(prop, "Width", "Bevel value/amount");
+       RNA_def_property_ui_text(prop, "Width", "Bevel amount");
+       RNA_def_property_update(prop, 0, "rna_Modifier_update");
+
+       prop = RNA_def_property(srna, "width_pct", PROP_FLOAT, PROP_FACTOR);
+       RNA_def_property_float_sdna(prop, NULL, "value");
+       RNA_def_property_range(prop, 0, FLT_MAX);
+       RNA_def_property_ui_range(prop, 0.0f, 100.0f, 5.0, 2);
+       RNA_def_property_ui_text(prop, "Width Percent", "Bevel amount for percentage method");
        RNA_def_property_update(prop, 0, "rna_Modifier_update");
 
        prop = RNA_def_property(srna, "segments", PROP_INT, PROP_NONE);
index 363cdb083d47d58a3d65c38d585d5e8760131671..6eee697a73978c7f22b331bc03c9568e0a90c789 100644 (file)
@@ -112,6 +112,7 @@ static Mesh *applyModifier(ModifierData *md, const ModifierEvalContext *ctx, Mes
        const bool vertex_only = (bmd->flags & MOD_BEVEL_VERT) != 0;
        const bool do_clamp = !(bmd->flags & MOD_BEVEL_OVERLAP_OK);
        const int offset_type = bmd->val_flags;
+       const float value = bmd->value;
        const int mat = CLAMPIS(bmd->mat, -1, ctx->object->totcol - 1);
        const bool loop_slide = (bmd->flags & MOD_BEVEL_EVEN_WIDTHS) == 0;
        const bool mark_seam = (bmd->edge_flags & MOD_BEVEL_MARK_SEAM);
@@ -192,7 +193,7 @@ static Mesh *applyModifier(ModifierData *md, const ModifierEvalContext *ctx, Mes
                harden_normals = false;
        }
 
-       BM_mesh_bevel(bm, bmd->value, offset_type, bmd->res, bmd->profile,
+       BM_mesh_bevel(bm, value, offset_type, bmd->res, bmd->profile,
                      vertex_only, bmd->lim_flags & MOD_BEVEL_WEIGHT, do_clamp,
                      dvert, vgroup, mat, loop_slide, mark_seam, mark_sharp,
                      harden_normals, face_strength_mode, mesh->smoothresh);