Fix #111174: dropped nodegroups from the asset browser show selector #111427

Merged
Philipp Oeser merged 3 commits from lichtwerk/blender:111174 into main 2023-08-24 15:12:08 +02:00

3 Commits

Author SHA1 Message Date
Philipp Oeser 0ef3d0d55b address review 2023-08-23 15:03:09 +02:00
Philipp Oeser d4eee7b0cd use more decriptive property name 2023-08-23 14:33:59 +02:00
Philipp Oeser 076bf4e55a Fix #111174: dropped nodegroups from the asset browser show selector
There was a difference between drag&drop assets from the asset browser
vs. using the search menu (in that doing it from the menu would
correctly hide the data-block selector on the nodegroup).

Since drag&drop in the Node Editor uses `NODE_OT_add_group` (not
`NODE_OT_add_group_asset` as the menu does), we have to add the hiding
here too (for this, an operator property is added which is set in
`node_group_drop_copy` if we are dropping an asset.

Alternatively, we could use `NODE_OT_add_group_asset`, too, but that
would require somehow setting the "asset" context pointer from the
dropbox copy function [how to do this wasnt obvious for me]. In that
case, we would need to set up a separate dropbox with appropriate poll
functions (so there would be one for asset groups and for the the other
groups).
2023-08-23 13:30:20 +02:00