Fix T84953: Incorrect tooltip for dragging collections

"Shift to parent" does not make any sense for collections since they
don't have parenting like objects. This commit just adds a simple check
for whether the first drag ID is an object before displaying
that part of the message.

Differential Revision: https://developer.blender.org/D10203
This commit is contained in:
Hans Goudey 2021-02-25 19:23:02 -06:00
parent b1b9671c93
commit 81e795e7f0
Notes: blender-bot 2023-02-14 06:54:28 +01:00
Referenced by issue #84953, Outliner: wrong tooltip for moving collections
1 changed files with 14 additions and 2 deletions

View File

@ -1213,11 +1213,23 @@ static bool collection_drop_poll(bContext *C,
*r_tooltip = TIP_("Move after collection");
}
break;
case TE_INSERT_INTO:
case TE_INSERT_INTO: {
tselem->flag |= TSE_DRAG_INTO;
changed = true;
*r_tooltip = TIP_("Move inside collection (Ctrl to link, Shift to parent)");
/* Check the type of the drag IDs to avoid the incorrect "Shift to parent"
* for collections. Checking the type of the first ID works fine here since
* all drag IDs are the same type. */
wmDragID *drag_id = (wmDragID *)drag->ids.first;
const bool is_object = (GS(drag_id->id->name) == ID_OB);
if (is_object) {
*r_tooltip = TIP_("Move inside collection (Ctrl to link, Shift to parent)");
}
else {
*r_tooltip = TIP_("Move inside collection (Ctrl to link)");
}
break;
}
}
}
if (changed) {