Blender Git Loki

Fix T86677: select grouped in node editor crashes without active node

This was reported for geometry nodes, but was true for all nodetrees
(e.g. after deleting the active node). Geometry node trees just made
this more obvious since they start without an active node to begin with.

Fix provided by @lone_noel, thx!

Maniphest Tasks: T86677

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

Fix T86548: Sculpt: Mask by Color tool not working

Caused by {rB2917f550caa9} which renamed the entry in the toolsystem,
but not the corresponding keymap.

Maniphest Tasks: T86548

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

LibOverride: Outliner: Tweak override creation from instancing empty.

Now behavior is similar to the one from 3DView: once override of the
collection is successfuly created, we remove the instancing empty from
the scene.

Fix 3DView not updating on some NC_ID notifiers.

Outliner uses a lot `NC_ID | NA_EDITED` e.g., which was not caught by
the View3D editor for update.

LibOverride: Outliner: Rename `Add...` to `Make...`.

Also more consistent with the name used for the 3DView operator.

LibOverride: Fix missing update after an override reset.

We need to brute-force reset IDs with `ID_RECALC_ALL` here...

Fix missing view3d updates after recent NC_SPACE notifier filters

Since {rB46aa70cb486d}, using `NC_SPACE | ND_SPACE_VIEW3D` as notifier is
restricted to space data as a reference. This was still used though for
RNA updates in other places (namely `rna_camera`, `rna_scene`,
`rna_animviz`), and passing NULL would automatically set the notifier
reference to the owner id. Above commit would happily filter these out,
leading to missing refreshes.

Now use more specific notifiers (in case of animviz a new
`ND_DRAW_ANIMVIZ` was added).

This was reported for Camera background images btw.
Fixes T86670.

Maniphest Tasks: T86670

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

UI: Dynamically increase data-block name button size if there's space

Data-block selectors (or other selectors using the
`UILayout.teamplate_search` or `UILayout.template_search_preview`)
in headers used a fixed size width for the name button. Often that meant
the text would be clipped to fit even though there was plenty of
whitespace surrounding the data-block selector.

Now the name button can increase in width with the contained string,
clamped to some arbitrary and quite big maximum (3 times the minimum
width).
We could further take the available space into account, to reduce the
need for scrolling in the header. But I don't think that is much of an
issue and the name clipping it would re-introduce would probably be more
annoying.

Fix T86594: Overrides: Possible collection "duplication".

Issue was actually in some Collection management code, a bit too eager
to add collection to the scene master one when it was not actually
needed.

LineArt: Add missing dna variable rename

Forgot to add this in: "Cleanup, LineArt: Rename LineartLine -> LineartEdge"
df280637952d35cfaa74e31f03a0d825b22eddf4

Cleanup, LineArt: Sample -> Resample

Clear up what sample length does by renaming the option and variables.

Cleanup: Remove unused variable in gpencil_add_lineart.c

LineArt: Expose the allow_duplication property for instanced objects.

It was missing from the UI.

Fix T86692: Start Line Art GP objects on frame 0

If playback starts before the first GP frame, we will get "ghost"
strokes.

LineArt: Do not calculate lines for objects not included

Only use non included objects for potential occlusion queries.
Don't calculate lines for them.

This fixes intersection lines appearing on objects not included by the
lineart modifier.

Cleanup, LineArt: Rename LineartLine -> LineartEdge

Also cleanup various other "line" variable names

GPencil: Fix unreported datablock type changed using Dopesheet box select

When doing a box selection in the dopesheet in the header area, the data block type was changed to annotation because the pointer was not usable and produced an unexpected result.

Thanks to @pullup for finding a way to reproduce the bug.

BLO: Functions for temporarily loading a single datablock

This introduces two functions to the blenloader module, here shown as
calls for brevity:

* `temp_lib_ctx = BLO_library_temp_load_id(real_main, blend_file_path, idcode, idname, reports);`
* Now the data in `temp_lib_ctx->temp_id` can be used (but ought not to
be not assigned to non-temp datablocks).
* `BLO_library_temp_free(temp_lib_ctx);`

The first loads a datablock from a blend file, and returns it as part of
a `struct TempLibraryContext`. This struct contains the temp-loaded ID,
as well as enough information to correctly free everything again.

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

Fix T86219: Compositor backdrop not using Color Management View
Transforms in certain cases

This was caused by wrong flag checking in {rB278011e44d43}, which just
seems to be a copy-paste error.

For example, enabeling 'Auto-Offset' in the View menu would lead to CM
being ignored.

Maniphest Tasks: T86219

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

Cleanup: use doxy sections for pose_transform.c