From fc59febb1b30f390e733e1c775b93d128dd84e34 Mon Sep 17 00:00:00 2001 From: Julian Eisel Date: Tue, 12 May 2020 19:32:52 +0200 Subject: UI/Animation: Don't change Graph Editor cursor from scrubbing region Do not execute the operator to change the Graph Editor cursor when changing the active frame through the scrubbing region. This is not what users expect, see T76599. Removes an explicit exception to not let the regular frame change operator execute in the Graph Editor. It was needed to let the cursor operator run instead, but the interplay between the two operators is now handled at keymap level. Fixes T76599. --- source/blender/editors/animation/anim_ops.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'source/blender/editors/animation/anim_ops.c') diff --git a/source/blender/editors/animation/anim_ops.c b/source/blender/editors/animation/anim_ops.c index 2db381dfc69..40cd368e02b 100644 --- a/source/blender/editors/animation/anim_ops.c +++ b/source/blender/editors/animation/anim_ops.c @@ -73,15 +73,9 @@ static bool change_frame_poll(bContext *C) * this shouldn't show up in 3D editor (or others without 2D timeline view) via search */ if (area) { - if (ELEM(area->spacetype, SPACE_ACTION, SPACE_NLA, SPACE_SEQ, SPACE_CLIP)) { + if (ELEM(area->spacetype, SPACE_ACTION, SPACE_NLA, SPACE_SEQ, SPACE_CLIP, SPACE_GRAPH)) { return true; } - else if (area->spacetype == SPACE_GRAPH) { - /* NOTE: Graph Editor has special version which does some extra stuff. - * No need to show the generic error message for that case though! - */ - return false; - } } CTX_wm_operator_poll_msg_set(C, "Expected an animation area to be active"); -- cgit v1.2.3