From 4c45f003ede5fdc1ca9da3f51a8b7d5764a0e0a2 Mon Sep 17 00:00:00 2001 From: Philippe Saint-Pierre Date: Mon, 23 Jul 2012 22:38:44 -0400 Subject: [PATCH] Tracker: Right clicking on Pose triggered Rename prompts Right clicking on a Pose to get the contextual menu would quite often trigger a rename action of that pose. Don't allow to rename a pose by releasing the secondary mouse button. --- src/kits/tracker/PoseView.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/kits/tracker/PoseView.cpp b/src/kits/tracker/PoseView.cpp index 2349d73631..0e7304bf52 100644 --- a/src/kits/tracker/PoseView.cpp +++ b/src/kits/tracker/PoseView.cpp @@ -6929,10 +6929,11 @@ BPoseView::MouseUp(BPoint where) int32 index; BPose* pose = FindPose(where, &index); - if (pose != NULL && fAllowPoseEditing) + uint32 lastButtons = Window()->CurrentMessage()->FindInt32("last_buttons"); + if (pose != NULL && fAllowPoseEditing && !fTrackRightMouseUp) pose->MouseUp(BPoint(0, index * fListElemHeight), this, where, index); - uint32 lastButtons = Window()->CurrentMessage()->FindInt32("last_buttons"); + // this handy field has been added by the tracking filter. // we need lastButtons for right button mouse-up tracking, // because there's currently no way to know wich buttons were