[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [51331] trunk/blender: Making " Jump to Keyframes" operator (for Action/Graph Editors) more obvious

[Joshua Leung]

Revision: 51331
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=51331
Author:   aligorith
Date:     2012-10-15 03:52:27 +0000 (Mon, 15 Oct 2012)
Log Message:
-----------
Making "Jump to Keyframes" operator (for Action/Graph Editors) more obvious

This operator used to be called "Jump to Frame". It basically takes the midpoint
(frame number and/or value) of selected keyframes, and positions the current
frame (or2d-cursor in Graph Editor) at this point.

The hotkey for this is now Ctrl-G (i.e. as it's similar to a "Goto Frame"
feature). It is also now in the Key menu instead of in the relatively obscure
View menu, even though it doesn't actually result in any keyframe edits taking
place.

(Also, fixed a typo/grammer issue with one of Remove Bone Group operator)

Modified Paths:
--------------
    trunk/blender/release/scripts/startup/bl_ui/space_dopesheet.py
    trunk/blender/release/scripts/startup/bl_ui/space_graph.py
    trunk/blender/source/blender/editors/armature/poseobject.c
    trunk/blender/source/blender/editors/space_action/action_edit.c
    trunk/blender/source/blender/editors/space_action/action_ops.c
    trunk/blender/source/blender/editors/space_graph/graph_edit.c
    trunk/blender/source/blender/editors/space_graph/graph_ops.c

Modified: trunk/blender/release/scripts/startup/bl_ui/space_dopesheet.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_ui/space_dopesheet.py	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/release/scripts/startup/bl_ui/space_dopesheet.py	2012-10-15 03:52:27 UTC (rev 51331)
@@ -165,7 +165,6 @@
         layout.operator("action.previewrange_set")
 
         layout.separator()
-        layout.operator("action.frame_jump")
         layout.operator("action.view_all")
         layout.operator("action.view_selected")
 
@@ -276,6 +275,9 @@
         layout.operator("action.keyframe_insert")
 
         layout.separator()
+        layout.operator("action.frame_jump")        
+        
+        layout.separator()
         layout.operator("action.duplicate_move")
         layout.operator("action.delete")
 

Modified: trunk/blender/release/scripts/startup/bl_ui/space_graph.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_ui/space_graph.py	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/release/scripts/startup/bl_ui/space_graph.py	2012-10-15 03:52:27 UTC (rev 51331)
@@ -96,7 +96,6 @@
         layout.operator("graph.previewrange_set")
 
         layout.separator()
-        layout.operator("graph.frame_jump")
         layout.operator("graph.view_all")
         layout.operator("graph.view_selected")
 
@@ -199,6 +198,9 @@
         layout.operator("graph.sound_bake")
 
         layout.separator()
+        layout.operator("graph.frame_jump")
+
+        layout.separator()
         layout.operator("graph.duplicate_move")
         layout.operator("graph.delete")
 

Modified: trunk/blender/source/blender/editors/armature/poseobject.c
===================================================================
--- trunk/blender/source/blender/editors/armature/poseobject.c	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/source/blender/editors/armature/poseobject.c	2012-10-15 03:52:27 UTC (rev 51331)
@@ -1399,7 +1399,7 @@
 	/* identifiers */
 	ot->name = "Remove Bone Group";
 	ot->idname = "POSE_OT_group_remove";
-	ot->description = "Removes the active bone group";
+	ot->description = "Remove the active bone group";
 	
 	/* api callbacks */
 	ot->exec = pose_group_remove_exec;

Modified: trunk/blender/source/blender/editors/space_action/action_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_action/action_edit.c	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/source/blender/editors/space_action/action_edit.c	2012-10-15 03:52:27 UTC (rev 51331)
@@ -1353,9 +1353,9 @@
 void ACTION_OT_frame_jump(wmOperatorType *ot)
 {
 	/* identifiers */
-	ot->name = "Jump to Frame";
+	ot->name = "Jump to Keyframes";
 	ot->idname = "ACTION_OT_frame_jump";
-	ot->description = "Set the current frame to the average frame of the selected keyframes";
+	ot->description = "Set the current frame to the average frame value of selected keyframes";
 	
 	/* api callbacks */
 	ot->exec = actkeys_framejump_exec;

Modified: trunk/blender/source/blender/editors/space_action/action_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_action/action_ops.c	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/source/blender/editors/space_action/action_ops.c	2012-10-15 03:52:27 UTC (rev 51331)
@@ -168,9 +168,8 @@
 	
 	
 	/* action_edit.c */
-	/* snap - current frame to selected keys */
-	// TODO: maybe since this is called jump, we're better to have it on <something>-J?
-	WM_keymap_add_item(keymap, "ACTION_OT_frame_jump", SKEY, KM_PRESS, KM_CTRL | KM_SHIFT, 0);
+	/* jump to selected keyframes */
+	WM_keymap_add_item(keymap, "ACTION_OT_frame_jump", GKEY, KM_PRESS, KM_CTRL, 0);
 		
 	/* menu + single-step transform */
 	WM_keymap_add_item(keymap, "ACTION_OT_snap", SKEY, KM_PRESS, KM_SHIFT, 0);

Modified: trunk/blender/source/blender/editors/space_graph/graph_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_graph/graph_edit.c	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/source/blender/editors/space_graph/graph_edit.c	2012-10-15 03:52:27 UTC (rev 51331)
@@ -1808,9 +1808,9 @@
 void GRAPH_OT_frame_jump(wmOperatorType *ot)
 {
 	/* identifiers */
-	ot->name = "Jump to Frame";
+	ot->name = "Jump to Keyframes";
 	ot->idname = "GRAPH_OT_frame_jump";
-	ot->description = "Set the current frame to the average frame of the selected keyframes";
+	ot->description = "Place the cursor on the midpoint of selected keyframes";
 	
 	/* api callbacks */
 	ot->exec = graphkeys_framejump_exec;

Modified: trunk/blender/source/blender/editors/space_graph/graph_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_graph/graph_ops.c	2012-10-15 03:36:22 UTC (rev 51330)
+++ trunk/blender/source/blender/editors/space_graph/graph_ops.c	2012-10-15 03:52:27 UTC (rev 51331)
@@ -346,9 +346,8 @@
 	
 	
 	/* graph_edit.c */
-	/* snap - current frame to selected keys */
-	// TODO: maybe since this is called jump, we're better to have it on <something>-J?
-	WM_keymap_add_item(keymap, "GRAPH_OT_frame_jump", SKEY, KM_PRESS, KM_CTRL | KM_SHIFT, 0);
+	/* jump to selected keyframes */
+	WM_keymap_add_item(keymap, "GRAPH_OT_frame_jump", GKEY, KM_PRESS, KM_CTRL, 0);
 		
 	/* menu + single-step transform */
 	WM_keymap_add_item(keymap, "GRAPH_OT_snap", SKEY, KM_PRESS, KM_SHIFT, 0);