[Bf-blender-cvs] [30cd1d10a95] master: Asset Browser: Remove catalog deletion confirm prompt

Julian Eisel noreply at git.blender.org
Tue Oct 12 15:51:43 CEST 2021


Commit: 30cd1d10a95a5e32b9e0daf1a24c41123f262c96
Author: Julian Eisel
Date:   Tue Oct 12 15:49:57 2021 +0200
Branches: master
https://developer.blender.org/rB30cd1d10a95a5e32b9e0daf1a24c41123f262c96

Asset Browser: Remove catalog deletion confirm prompt

This confirmation popup was added when deletion was a destructive action that
would be written to disk immediately, with no way to undo. Now we only write
such changes to disk on .blend file save, plus there's undo/redo support for
catalog edits now. In such cases confirmation popups should be avoided.

===================================================================

M	source/blender/editors/asset/intern/asset_ops.cc

===================================================================

diff --git a/source/blender/editors/asset/intern/asset_ops.cc b/source/blender/editors/asset/intern/asset_ops.cc
index 3d035396961..090cc53eb3b 100644
--- a/source/blender/editors/asset/intern/asset_ops.cc
+++ b/source/blender/editors/asset/intern/asset_ops.cc
@@ -450,7 +450,6 @@ static void ASSET_OT_catalog_delete(struct wmOperatorType *ot)
 
   /* api callbacks */
   ot->exec = asset_catalog_delete_exec;
-  ot->invoke = WM_operator_confirm;
   ot->poll = asset_catalog_operator_poll;
 
   RNA_def_string(ot->srna, "catalog_id", nullptr, 0, "Catalog ID", "ID of the catalog to delete");



More information about the Bf-blender-cvs mailing list