Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Tracker: Add dragged poses to the drag message in the correct order

Browse Source 
The function EachPoseAndModel iterates through poses in reverse order.
This fixes #12333.

Signed-off-by: Augustin Cavalier <waddlesplash@gmail.com>
This commit is contained in:
Markus Himmel 2015-12-17 12:01:15 +01:00 committed by Augustin Cavalier
parent 33840df504
commit a89ac5a653
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/kits/tracker/PoseView.cpp
View File

@ -7496,7 +7496,7 @@ BPoseView::WasDoubleClick(const BPose* pose, BPoint point, int32 buttons)
static void
AddPoseRefToMessage(BPose*, Model* model, BMessage* message)
AddPoseRefToMessage(Model* model, BMessage* message)
{
// Make sure that every file added to the message has its
// MIME type set.
@ -7536,7 +7536,12 @@ BPoseView::DragSelectedPoses(const BPose* pose, BPoint clickPoint)
// add Tracker token so that refs received recipients can script us
message.AddMessenger("TrackerViewToken", BMessenger(this));
EachPoseAndModel(fSelectionList, &AddPoseRefToMessage, &message);
// cannot use EachPoseAndModel here, because that iterates the selected
// poses in reverse order
for (int32 index = 0; index < fSelectionList->CountItems(); index++) {
AddPoseRefToMessage(fSelectionList->ItemAt(index)->TargetModel(),
&message);
}
// make sure button is still down
uint32 buttons;