[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [18724] branches/blender2.5/blender/source /blender/editors: Animation Editors: Tweaking/Refresh Issues

[Joshua Leung]

Revision: 18724
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=18724
Author:   aligorith
Date:     2009-01-28 23:51:42 +0100 (Wed, 28 Jan 2009)

Log Message:
-----------
Animation Editors: Tweaking/Refresh Issues

* Added 'select-tweak' functionality to DopeSheet
* Added proper notifiers for Graph Editor updates 

Modified Paths:
--------------
    branches/blender2.5/blender/source/blender/editors/animation/anim_deps.c
    branches/blender2.5/blender/source/blender/editors/space_action/action_ops.c
    branches/blender2.5/blender/source/blender/editors/space_action/action_select.c

Modified: branches/blender2.5/blender/source/blender/editors/animation/anim_deps.c
===================================================================
--- branches/blender2.5/blender/source/blender/editors/animation/anim_deps.c	2009-01-28 22:36:34 UTC (rev 18723)
+++ branches/blender2.5/blender/source/blender/editors/animation/anim_deps.c	2009-01-28 22:51:42 UTC (rev 18724)
@@ -145,6 +145,8 @@
 	/* types of notifiers to send, depends on the editor context */
 	switch (ac->datatype) {
 		case ANIMCONT_DOPESHEET: /* dopesheet */
+		case ANIMCONT_FCURVES: /* fcurve editor */
+		case ANIMCONT_DRIVERS: /* drivers editor */	// XXX probably this will need separate handling, since these are part of dependency system 
 		{
 			/* what action was taken */
 			switch (data_changed) {
@@ -153,8 +155,9 @@
 					// XXX what about other cases? maybe we need general ND_KEYFRAMES or ND_ANIMATION?
 					WM_event_add_notifier(C, NC_OBJECT|ND_TRANSFORM, NULL);
 					break;
-				//case ANIM_CHANGED_KEYFRAMES_SELECT:	// XXX what to do here?
-				//	break;
+				case ANIM_CHANGED_KEYFRAMES_SELECT:	// XXX what to do here?
+					WM_event_add_notifier(C, NC_SCENE, NULL);
+					break;
 				case ANIM_CHANGED_CHANNELS:
 					// XXX err... check available datatypes in dopesheet first?
 					// FIXME: this currently doesn't work (to update own view)

Modified: branches/blender2.5/blender/source/blender/editors/space_action/action_ops.c
===================================================================
--- branches/blender2.5/blender/source/blender/editors/space_action/action_ops.c	2009-01-28 22:36:34 UTC (rev 18723)
+++ branches/blender2.5/blender/source/blender/editors/space_action/action_ops.c	2009-01-28 22:51:42 UTC (rev 18724)
@@ -138,6 +138,9 @@
 		/* auto-set range */
 	WM_keymap_add_item(keymap, "ACT_OT_set_previewrange", PKEY, KM_PRESS, KM_CTRL|KM_ALT, 0);
 	WM_keymap_add_item(keymap, "ACT_OT_view_all", HOMEKEY, KM_PRESS, 0, 0);
+	
+	/* generates event, needs to be after select to work */
+	WM_keymap_add_item(keymap, "WM_OT_tweak_gesture", SELECTMOUSE, KM_PRESS, 0, 0);
 		
 	/* transform system */
 	transform_keymap_for_space(wm, keymap, SPACE_ACTION);

Modified: branches/blender2.5/blender/source/blender/editors/space_action/action_select.c
===================================================================
--- branches/blender2.5/blender/source/blender/editors/space_action/action_select.c	2009-01-28 22:36:34 UTC (rev 18723)
+++ branches/blender2.5/blender/source/blender/editors/space_action/action_select.c	2009-01-28 22:51:42 UTC (rev 18724)
@@ -1068,7 +1068,8 @@
 	/* set notifier tha things have changed */
 	ANIM_animdata_send_notifiers(C, &ac, ANIM_CHANGED_BOTH);
 	
-	return OPERATOR_FINISHED;
+	/* for tweak grab to work */
+	return OPERATOR_FINISHED|OPERATOR_PASS_THROUGH;
 }
  
 void ACT_OT_keyframes_clickselect (wmOperatorType *ot)