[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [55331] trunk/blender: Some more assorted fixes

Joshua Leung aligorith at gmail.com
Sat Mar 16 06:59:00 CET 2013


Revision: 55331
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=55331
Author:   aligorith
Date:     2013-03-16 05:58:59 +0000 (Sat, 16 Mar 2013)
Log Message:
-----------
Some more assorted fixes
* More duplicate/wrong comments (copy+paste errors)
* Brough Calculate/Clear Motion Path UI for bones more in line with the Object
version

Modified Paths:
--------------
    trunk/blender/release/scripts/startup/bl_ui/space_view3d_toolbar.py
    trunk/blender/source/blender/editors/space_nla/nla_channels.c
    trunk/blender/source/blender/editors/space_nla/nla_edit.c

Modified: trunk/blender/release/scripts/startup/bl_ui/space_view3d_toolbar.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_ui/space_view3d_toolbar.py	2013-03-16 05:48:46 UTC (rev 55330)
+++ trunk/blender/release/scripts/startup/bl_ui/space_view3d_toolbar.py	2013-03-16 05:58:59 UTC (rev 55331)
@@ -472,8 +472,9 @@
 
         col = layout.column(align=True)
         col.label(text="Motion Paths:")
-        col.operator("pose.paths_calculate", text="Calculate Paths")
-        col.operator("pose.paths_clear", text="Clear Paths")
+        row = col.row(align=True)
+        row.operator("pose.paths_calculate", text="Calculate")
+        row.operator("pose.paths_clear", text="Clear")
 
         draw_repeat_tools(context, layout)
 

Modified: trunk/blender/source/blender/editors/space_nla/nla_channels.c
===================================================================
--- trunk/blender/source/blender/editors/space_nla/nla_channels.c	2013-03-16 05:48:46 UTC (rev 55330)
+++ trunk/blender/source/blender/editors/space_nla/nla_channels.c	2013-03-16 05:58:59 UTC (rev 55331)
@@ -469,7 +469,8 @@
 	return added;
 }
 
-/* Add Tracks exec() */
+/* ----- */
+
 static int nlaedit_add_tracks_exec(bContext *C, wmOperator *op)
 {
 	bAnimContext ac;

Modified: trunk/blender/source/blender/editors/space_nla/nla_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_nla/nla_edit.c	2013-03-16 05:48:46 UTC (rev 55330)
+++ trunk/blender/source/blender/editors/space_nla/nla_edit.c	2013-03-16 05:58:59 UTC (rev 55331)
@@ -1627,7 +1627,7 @@
 	
 	/* api callbacks */
 	ot->exec = nlaedit_sync_actlen_exec;
-	ot->poll = ED_operator_nla_active; // XXX: is this satisfactory... probably requires a check for active strip...
+	ot->poll = nlaop_poll_tweakmode_off;
 	
 	/* flags */
 	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
@@ -2102,8 +2102,10 @@
 		BKE_report(op->reports, RPT_ERROR, "No F-Modifiers available to be copied");
 		return OPERATOR_CANCELLED;
 	}
-	else
+	else {
+		/* no updates needed - copy is non-destructive operation */
 		return OPERATOR_FINISHED;
+	}
 }
  
 void NLA_OT_fmodifier_copy(wmOperatorType *ot)
@@ -2157,8 +2159,6 @@
 	
 	/* successful or not? */
 	if (ok) {
-		/* set notifier that things have changed */
-		/* set notifier that things have changed */
 		WM_event_add_notifier(C, NC_ANIMATION | ND_NLA | NA_EDITED, NULL);
 		return OPERATOR_FINISHED;
 	}




More information about the Bf-blender-cvs mailing list