Welcome to mirror list, hosted at ThFree Co, Russian Federation.

git.blender.org/blender.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoshua Leung <aligorith@gmail.com>2017-11-19 15:00:27 +0300
committerJoshua Leung <aligorith@gmail.com>2017-11-19 15:57:21 +0300
commitc5f59f5854732f7d2c2020ca7ec67f1eaa043b78 (patch)
tree03daf522dfdc68ddf3dfb460ebbc7d2488ef0968 /source/blender/makesrna
parentc01ffefb687260c11b3eae8411e5af051f0ce10a (diff)
Tweaks to make it easier to update bone motion paths (without the active bone needing to have any paths itself)
This commit introduces the following changes: * Modified the poll callback on the "Update Paths" operator for bones so that it only checks if there are bones that have motion paths (instead of checking whether the active bone has paths). This makes it easier to update paths without having to first select one that has them - useful when the paths are all on hidden/hard-to-select bones. * Add a readonly property, "has_motion_paths" to the animviz.motion_path RNA struct, providing easier access to the internal flag used above. This makes it possible for the UI to display the "Update" button without having to check various bones for motion paths. Notes: * The flag being used in these changes already existed, and was only really intended for internal use. However, since it was already used in many places for determining if auto-update of all bone paths was needed (e.g. after certain editing ops), it should be safe to use here too. * The update_paths operator currently bakes all paths when activated, so there's currently no loss of functionality with changing to not checking if the active bone has any paths (e.g. we couldn't only update the active bone only either). That is still listed as a todo in the code.
Diffstat (limited to 'source/blender/makesrna')
-rw-r--r--source/blender/makesrna/intern/rna_animviz.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/source/blender/makesrna/intern/rna_animviz.c b/source/blender/makesrna/intern/rna_animviz.c
index 0ae07874853..2ba067629a9 100644
--- a/source/blender/makesrna/intern/rna_animviz.c
+++ b/source/blender/makesrna/intern/rna_animviz.c
@@ -174,6 +174,7 @@ static void rna_def_animviz_motion_path(BlenderRNA *brna)
RNA_def_property_ui_text(prop, "Use Bone Heads",
"For PoseBone paths, use the bone head location when calculating this path");
+ /* FIXME: Motion Paths are not currently editable... */
prop = RNA_def_property(srna, "is_modified", PROP_BOOLEAN, PROP_NONE);
RNA_def_property_boolean_sdna(prop, NULL, "flag", MOTIONPATH_FLAG_EDIT);
RNA_def_property_ui_text(prop, "Edit Path", "Path is being edited");
@@ -364,6 +365,12 @@ static void rna_def_animviz_paths(BlenderRNA *brna)
"(only for 'Around Current Frame' Onion-skinning method)");
RNA_def_property_update(prop, NC_SPACE | ND_SPACE_VIEW3D, NULL); /* XXX since this is only for 3d-view drawing */
+
+ /* Readonly Property - Do any motion paths exist/need updating? (Mainly for bone paths) */
+ prop = RNA_def_property(srna, "has_motion_paths", PROP_BOOLEAN, PROP_NONE);
+ RNA_def_property_boolean_sdna(prop, NULL, "path_bakeflag", MOTIONPATH_BAKE_HAS_PATHS);
+ RNA_def_property_clear_flag(prop, PROP_EDITABLE); /* NOTE: This is really an internal state var for convenience, so don't allow edits! */
+ RNA_def_property_ui_text(prop, "Has Motion Paths", "Are there any bone paths that will need updating (read-only)");
}
/* --- */