Blender Git Loki

UI: Support tooltips for superimposed icons

In a couple of places in the UI, we show superimposed icons on buttons
to execute an operation (called "Extra Icons" internally). Hovering them
would show the tooltip of the underlying button, which is misleading and
confusing.
There are cases where it's not obvious what an icon does, so a tooltip
would be quite useful here. It's likely we are going to use superimposed
icons in more places in the future, e.g. see D11890.

The extra icon basically acts as an override for the button in the
tooltip code.

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

Reviewed by: Campbell Barton

Py doc: document msgbus subscriptions clearing on blendfile load

Document that `bpy.msgbus.subscribe_rna()`-registered messagebus
subscriptions will be cleared whenever a new blend file is loaded.

Passing `options={'PERSISTENT'}` has no influence on this behaviour.

UI/Assets: Disable undo for tree item buttons

Disables undo for:
* The tree row collapsing - which doesn't make sense to undo, isn't
supported by the undo system, and just triggers the confirmation
prompt when closing the file.
* Renaming items - While this may make sense in some cases, users of the
tree-view API can explicitly do an undo push. For asset catalogs it's
not supported.

Cleanup: asset catalogs, move functions to their siblings

Moved function definitions around so that all members of a class are
next to each other. Previously some functions of one class were sitting
between functions of another class.

No functional changes.

VSE: Implement the bounding box (xform) tool in the seq preview window

Make the "xform" tool/gizmo available for strip transformations in the
sequencer preview window.

Because of the amount of hacks needed to make the gizmo work nicely with
multiple strips at the same time, it was decided to only show the
translate gizmo when multiple strips are selected.

This is because the transforms with multiple strips would appear buggy
because of our lack of shearing support in the transform system.
There is also currently no way to properly sync the gizmo drawing with
the transform when using multiple strips.

Reviewed By: Richard Antalik, Campbell Barton

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

Asset Catalogs: Fix unit test on Windows

Force native slashes before comparing paths; on Windows mixing
separators caused a unit test to fail.

No functional changes.

Cleanup: Else after return in Cycles

Cleanup: Explicit specifier for single argument constructor

Fix T91190: Remove gaps operator not working

Caused by mistake in f49d438ced7c - `SEQ_query_all_strips()` is not
meant to be recursive.

Sequencer: hide gizmos & cursor during scrubbing & playback

This was distracting prevented easily viewing an animation.

Fix crash switching scenes with sequencer gizmos displayed

Sequencer: add option to toggle gizmos

Use shortcut matching the 3D view & popover in the header

Cleanup: spelling

Sequencer: only show the 2D cursor with overlays enabled

Also hide when displaying scopes.

Fix sequencer preview/strip checks

- Drawing annotations used a deprecated value to detect
if drawing was supported.
- ED_space_sequencer_check_show_strip was checking the preview
image type (which doesn't impact strip display).

Signed-off-by: Campbell Barton <ideasman42@gmail.com>

Cleanup: remove deprecated SEQ_DRAW_SEQUENCE value

While drawing cleared this value (as part of temporary fix from 2009),
this was still being checked until recently.

Remove this value in versioning code.

Also clear unused text space flag.

Cleanup: remove redundant cursor copy, print

Cleanup: remove unnecessary data from LocalIntersection

Fix Cycles random walk SSS differences between CPU and GPU

The Embree logic did not match the GPU.

Cleanup: Rename enum values

This makes the diff for adding a new version of the attribute transfer
node slightly smaller.