blender
/
blender
129
398
Fork 572
Code Issues 6.2k Pull Requests 748 Projects 19 Wiki Activity

Outliner: Support multiple-objects drag'n'drop into collections 

We now can drag multiple objects at once in the outliner. You we restricted to
working within a single outliner. Be sure to drag from the objects name, not
from its icon (otherwise it will try to parent it).

We don't use the same drag'n'drop system as IDs here. Which although I dislike
allowed for this solution to be local, and not dependent on the entire
drag'n'drop system of Blender.

This is a feature Andy Goralczyk has requested a long time ago.
Kudos for him for his request.
This commit is contained in:
Dalai Felinto 2018-01-05 18:09:58 -02:00
parent 9d134a22c2
commit fb4cd136a7
Notes: blender-bot 2023-02-14 10:09:24 +01:00 
Referenced by commit ae12424298, Outliner/Collections: Fix objects disappearing when moving to collections
3 changed files with 64 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
source/blender/editors/space_outliner/outliner_intern.h
View File

@ -69,6 +69,7 @@ typedef enum TreeTraversalAction {
* Callback type for reinserting elements at a different position, used to allow user customizable element order.
*/
typedef void (*TreeElementReinsertFunc)(struct Main *bmain,
struct SpaceOops *soops,
struct TreeElement *insert_element,
struct TreeElement *insert_handle, TreeElementInsertType action);
/**

6
source/blender/editors/space_outliner/outliner_ops.c
View File

@ -164,7 +164,7 @@ static void outliner_item_drag_handle(
te_dragged->drag_data->insert_handle = te_insert_handle;
}
static bool outliner_item_drag_drop_apply(Main *bmain, TreeElement *dragged_te)
static bool outliner_item_drag_drop_apply(Main *bmain, SpaceOops *soops, TreeElement *dragged_te)
{
TreeElement *insert_handle = dragged_te->drag_data->insert_handle;
TreeElementInsertType insert_type = dragged_te->drag_data->insert_type;
@ -178,7 +178,7 @@ static bool outliner_item_drag_drop_apply(Main *bmain, TreeElement *dragged_te)
/* call of assert above should not have changed insert_handle and insert_type at this point */
BLI_assert(dragged_te->drag_data->insert_handle == insert_handle &&
dragged_te->drag_data->insert_type == insert_type);
dragged_te->reinsert(bmain, dragged_te, insert_handle, insert_type);
dragged_te->reinsert(bmain, soops, dragged_te, insert_handle, insert_type);
return true;
}
@ -198,7 +198,7 @@ static int outliner_item_drag_drop_modal(bContext *C, wmOperator *op, const wmEv
switch (event->type) {
case EVT_MODAL_MAP:
if (event->val == OUTLINER_ITEM_DRAG_CONFIRM) {
if (outliner_item_drag_drop_apply(bmain, te_dragged)) {
if (outliner_item_drag_drop_apply(bmain, soops, te_dragged)) {
skip_rebuild = false;
}
retval = OPERATOR_FINISHED;

75
source/blender/editors/space_outliner/outliner_tree.c
View File

@ -387,12 +387,38 @@ static void outliner_add_scene_contents(SpaceOops *soops, ListBase *lb, Scene *s
#endif
}
static void outliner_object_reorder(
Main *UNUSED(bmain),
TreeElement *insert_element, TreeElement *insert_handle, TreeElementInsertType action)
struct ObjectsSelectedData {
ListBase objects_selected_array;
};
static TreeTraversalAction outliner_find_selected_objects(TreeElement *te, void *customdata)
{
struct ObjectsSelectedData *data = customdata;
TreeStoreElem *tselem = TREESTORE(te);
if (ELEM(tselem->type, TSE_LAYER_COLLECTION, TSE_SCENE_COLLECTION)) {
return TRAVERSE_CONTINUE;
}
if (tselem->type || (tselem->id == NULL) || (GS(tselem->id->name) != ID_OB)) {
return TRAVERSE_SKIP_CHILDS;
}
BLI_addtail(&data->objects_selected_array, BLI_genericNodeN(te));
return TRAVERSE_CONTINUE;
}
/**
* Move objects from a collection to another.
* We ignore the original object being inserted, we used it for polling only.
* Instead we move all the selected objects around.
*/
static void outliner_object_reorder(
Main *bmain, SpaceOops *soops,
TreeElement *insert_element,
TreeElement *insert_handle, TreeElementInsertType action)
{
TreeStoreElem *tselem_insert = TREESTORE(insert_element);
Object *ob = (Object *)tselem_insert->id;
SceneCollection *sc = outliner_scene_collection_from_tree_element(insert_handle);
SceneCollection *sc_ob_parent = NULL;
ID *id = insert_handle->store_elem->id;
@ -400,19 +426,36 @@ static void outliner_object_reorder(
BLI_assert(action == TE_INSERT_INTO);
UNUSED_VARS_NDEBUG(action);
/* find parent scene-collection of object */
if (insert_element->parent) {
for (TreeElement *te_ob_parent = insert_element->parent; te_ob_parent; te_ob_parent = te_ob_parent->parent) {
if (ELEM(TREESTORE(te_ob_parent)->type, TSE_SCENE_COLLECTION, TSE_LAYER_COLLECTION)) {
sc_ob_parent = outliner_scene_collection_from_tree_element(te_ob_parent);
break;
struct ObjectsSelectedData data = {
.objects_selected_array = {NULL, NULL},
};
/* Make sure we include the originally inserted element as well. */
TREESTORE(insert_element)->flag |= TSE_SELECTED;
outliner_tree_traverse(soops, &soops->tree, 0, TSE_SELECTED, outliner_find_selected_objects, &data);
LINKLIST_FOREACH(LinkData *, link, &data.objects_selected_array) {
TreeElement *ten_selected = (TreeElement *)link->data;
/* Find parent scene-collection of object. */
if (ten_selected->parent) {
for (TreeElement *te_ob_parent = ten_selected->parent; te_ob_parent; te_ob_parent = te_ob_parent->parent) {
if (ELEM(TREESTORE(te_ob_parent)->type, TSE_SCENE_COLLECTION, TSE_LAYER_COLLECTION)) {
sc_ob_parent = outliner_scene_collection_from_tree_element(te_ob_parent);
break;
}
}
}
else {
sc_ob_parent = BKE_collection_master(id);
}
Object *ob = (Object *)TREESTORE(ten_selected)->id;
BKE_collection_object_move(id, sc, sc_ob_parent, ob);
}
else {
sc_ob_parent = BKE_collection_master(id);
}
BKE_collection_object_move(id, sc, sc_ob_parent, ob);
BLI_freelistN(&data.objects_selected_array);
DEG_relations_tag_update(bmain);
WM_main_add_notifier(NC_SCENE | ND_LAYER, NULL);
}
static bool outliner_object_reorder_poll(
@ -1358,6 +1401,7 @@ static void outliner_add_orphaned_datablocks(Main *mainvar, SpaceOops *soops)
static void outliner_layer_collections_reorder(
Main *bmain,
SpaceOops *UNUSED(soops),
TreeElement *insert_element, TreeElement *insert_handle, TreeElementInsertType action)
{
LayerCollection *lc_insert = insert_element->directdata;
@ -1419,6 +1463,7 @@ static void outliner_add_collections_act_layer(SpaceOops *soops, Scene *scene, V
static void outliner_scene_collections_reorder(
Main *bmain,
SpaceOops *UNUSED(soops),
TreeElement *insert_element, TreeElement *insert_handle, TreeElementInsertType action)
{
SceneCollection *sc_insert = insert_element->directdata;