diff options
author | Julian Eisel <julian@blender.org> | 2021-10-05 15:25:40 +0300 |
---|---|---|
committer | Julian Eisel <julian@blender.org> | 2021-10-05 17:10:27 +0300 |
commit | dbe3981b0a805c1a40f42f57dc7ccc3d28270fda (patch) | |
tree | c646101190ece7bf46ee025b79c261b792abb08e /source/blender/editors/interface/tree_view.cc | |
parent | 9a0850c8c25ea0c28f6ac313f076fd6a8563d0b4 (diff) |
Cleanup: Better way to pass activate callbacks to Tree-View items
The `ui::BasicTreeViewItem` took a function-like object to execute on
item activation via the constructor. This was mainly intended to be used
with lambdas. However, it's confusing to just have this lambda there,
with no indication of what it's for (activation).
Instead, assign the function-like object via an explicit `on_activate()`
function.
Diffstat (limited to 'source/blender/editors/interface/tree_view.cc')
-rw-r--r-- | source/blender/editors/interface/tree_view.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/source/blender/editors/interface/tree_view.cc b/source/blender/editors/interface/tree_view.cc index 202e3dba2ab..63b12d4fc89 100644 --- a/source/blender/editors/interface/tree_view.cc +++ b/source/blender/editors/interface/tree_view.cc @@ -278,8 +278,7 @@ uiLayout *TreeViewLayoutBuilder::current_layout() const /* ---------------------------------------------------------------------- */ -BasicTreeViewItem::BasicTreeViewItem(StringRef label, BIFIconID icon_, ActivateFn activate_fn) - : icon(icon_), activate_fn_(activate_fn) +BasicTreeViewItem::BasicTreeViewItem(StringRef label, BIFIconID icon_) : icon(icon_) { label_ = label; } @@ -330,6 +329,11 @@ void BasicTreeViewItem::on_activate() } } +void BasicTreeViewItem::on_activate(ActivateFn fn) +{ + activate_fn_ = fn; +} + BIFIconID BasicTreeViewItem::get_draw_icon() const { if (icon) { |