[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [13610] trunk/blender/source/blender/src: Ctrl-PageUp/ Down in 3D-view now behaves the same as for the Timeline for consistency.

[Joshua Leung]

Revision: 13610
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=13610
Author:   aligorith
Date:     2008-02-08 21:15:07 +0100 (Fri, 08 Feb 2008)

Log Message:
-----------
Ctrl-PageUp/Down in 3D-view now behaves the same as for the Timeline for consistency. The old behaviour for this (jumping between ob-ipo keyframes (Draw/Show Keys)) was not used that often, and has now been remapped to Shift-PageUp/Down

Modified Paths:
--------------
    trunk/blender/source/blender/src/header_info.c
    trunk/blender/source/blender/src/space.c

Modified: trunk/blender/source/blender/src/header_info.c
===================================================================
--- trunk/blender/source/blender/src/header_info.c	2008-02-08 15:47:47 UTC (rev 13609)
+++ trunk/blender/source/blender/src/header_info.c	2008-02-08 20:15:07 UTC (rev 13610)
@@ -111,6 +111,7 @@
 #include "BSE_node.h"
 #include "BSE_sequence.h"
 #include "BSE_edit.h"
+#include "BSE_time.h"
 
 #include "IMB_imbuf_types.h"
 
@@ -1639,23 +1640,23 @@
 		if (!ob) error("Select an object before showing and selecting its keyframes");
 		else select_select_keys();
 			break;
-		case 3:
+	case 3:
 		/* select next keyframe */
 		if (!ob) error("Select an object before selecting its next keyframe");
 		else nextkey_obipo(1);
 			break;
-		case 4:
+	case 4:
 		/* select previous keyframe */
 		if (!ob) error("Select an object before selecting its previous keyframe");
 		else nextkey_obipo(-1);
 		break;
 		case 5:
-		/* next keyframe */
+	/* next keyframe */
 		if (!ob) error("Select an object before going to its next keyframe");
 		else movekey_obipo(1);
 			break;
 		case 6:
-		/* previous keyframe */
+	/* previous keyframe */
 		if (!ob) error("Select an object before going to its previous keyframe");
 		else movekey_obipo(-1);
 		break;
@@ -1691,6 +1692,14 @@
 		CFRA= SFRA;
 		update_for_newframe();
 		break;
+	case 13: 
+		/* previous keyframe */
+		nextprev_timeline_key(-1);
+		break;
+	case 14:
+		/* next keyframe */
+		nextprev_timeline_key(1);
+		break;
 	}
 	allqueue(REDRAWINFO, 0);
 }
@@ -1713,9 +1722,12 @@
 
 	uiDefBut(block, SEPR, 0, "",				0, yco-=6, menuwidth, 6, NULL, 0.0, 0.0, 0, 0, "");
 
-	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Next Keyframe|Ctrl PageUp",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 5, "");
-	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Previous Keyframe|Ctrl PageDown",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 6, "");
+	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Next Ob-Keyframe|Shift PageUp",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 5, "");
+	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Previous Ob-Keyframe|Shift PageDown",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 6, "");
 
+	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Next Keyframe|Ctrl PageUp",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 13, "");
+	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Previous Keyframe|Ctrl PageDown",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 14, "");
+	
 	uiDefBut(block, SEPR, 0, "",				0, yco-=6, menuwidth, 6, NULL, 0.0, 0.0, 0, 0, "");
 
 	uiDefIconTextBut(block, BUTM, 1, ICON_BLANK1, "Next Frame|RightArrow",	0, yco-=20, menuwidth, 19, NULL, 0.0, 0.0, 1, 7, "");

Modified: trunk/blender/source/blender/src/space.c
===================================================================
--- trunk/blender/source/blender/src/space.c	2008-02-08 15:47:47 UTC (rev 13609)
+++ trunk/blender/source/blender/src/space.c	2008-02-08 20:15:07 UTC (rev 13610)
@@ -2648,16 +2648,21 @@
 				break;
 			
 			case PAGEUPKEY:
-				if(G.qual==LR_CTRLKEY)
+				if (G.qual==LR_CTRLKEY)
+					nextprev_timeline_key(1);
+				else if (G.qual==LR_SHIFTKEY)
 					movekey_obipo(1);
-				else if((G.qual==0))
+				else if (G.qual==0)
 					nextkey_obipo(1);	/* in editipo.c */
+					
 				break;
 
 			case PAGEDOWNKEY:
-				if(G.qual==LR_CTRLKEY)
+				if (G.qual==LR_CTRLKEY)
+					nextprev_timeline_key(-1);
+				else if (G.qual==LR_SHIFTKEY)
 					movekey_obipo(-1);
-				else if((G.qual==0))
+				else if (G.qual==0)
 					nextkey_obipo(-1);
 				break;