Blender Git Loki

Fix T92150: Incorrect invert in Delete Geometry node

The selection was inverted when deleting points from a spline.

Differential Revision: https://developer.blender.org/D12840

Revert "GHOST: Unify behavior of offscreen context creation"

Commited by mistake

This reverts commit 6535779c92b90035870047f178cf3eff95f0bdf0.

Fix dragging objects from Outliner to 3D View broken

A dragged & dropped wouldn't be duplicated anymore, it would just be moved to
the drop position.

Caused by c8fcea0c33ef.

Cleanup: use `nullptr` instead of `NULL` in C++ code.

Fix T91981: Crash when using operators that needs scene depth

There was a double lock in the object depth drawing function.
Also the texture read was not reading the texture with the right
format. Now it needs a conversion.

Fix T91981 Particle Edit make Blender Crash
Fix T92006 Light spot interactively point can't use

GHOST: Unify behavior of offscreen context creation

This makes sure the previously bound context is restored after creating a
new context. This follows what is already happening on windows.

All system backend are patched.

This also removes the goto and some code duplication.

Differential Revision: https://developer.blender.org/D12455

Fix warning about deleted default constructor declared as default

Since `AssetCatalogPath` isn't default constructible (unlike the
previous `CatalogPath`, alias `std::string`), `AssetCatalog` isn't
default constructible either. But its default constructor is declared
with `= default` which Apple Clang was warning about.

Differential Revision: https://developer.blender.org/D12714

Reviewed by: Sybren St�vel

Address warning about breaking copy elision of temporary object

Using `std::move()` on temporary objects prevents copy elision done by
compilers. Apple Clang warns about this.

Generally `std::move()` should be used sparingly:
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines

Cleanup: Typo in comment.

Cleanup: Attempt to fix benign macOS compile warnings

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.

Asset Catalogs Undo: add Ctrl+Z and Ctrl+Shift+Z as hotkeys for undo/redo

Allow undo/redo of asset catalog edits with Ctrl+Z/Ctrl+Shift+Z. These
keys are registered in the 'screen' keymap, so that they can take
priority over the global undo/redo operators.

Updated both Blender Default and Industry Compatible keymaps.

Asset Catalogs: mark 'undo push' operator as internal

Mark the `ASSET_OT_catalog_undo_push` operator as internal, as it's not
meant for artists to use directly.

Asset Catalog Undo: send notifiers to redraw the catalog tree

Send notifiers to redraw the catalog tree after undo/redo.

Asset Catalogs: expose undo/redo operators to UI

Ensure that catalog operations create an undo snapshot, and show
undo/redo operators in the asset browser.

A hidden operator `ASSET_OT_catalog_undo_push` is also added such that
add-ons can also set undo snapshots if they need.

Cleanup: asset catalogs, rename store_undo_snapshot to undo_push

Rename `bke::AssetCatalogService::store_undo_snapshot` to `undo_push`.
This makes the function named the same way as the global Blender "undo
push" function.

No functional changes.

Fix: Incorrect error message in set spline resolution node

Caused by my own refactoring before committing the patch.

Merge branch 'master' into asset-greasepencil

Asset Catalogs: undo stack for catalog edits

Add an undo stack for catalog edits. This only implements the backend,
no operators or UI yet.

A bunch of `this->xxx` has been replaced by `catalog_collection_->xxx`.
Things are getting a bit long, and the class is turning into a god
object; refactoring the class is tracked in T92114.

Reviewed By: Severin

Maniphest Tasks: T92047

Differential Revision: https://developer.blender.org/D12825

Assets: hide low-level catalog info behind "developer extra's"

Hide the catalog UUID and the catalog simple name from the Asset Browser
side-panel, unless "Developer Extra's" is enabled in the user prefs.