Blender Git Loki

Cleanup: spelling

Sculpt: fix pose brush flood fill with symmetry

Cleanup: move eIconSizes, ID_Type enums into own file

This avoids adding DNA_ID.h into other headers, recently changed in
cfa48c84d06ca8197f86b6d3ceef8a2c7c311a82

Note that other enums could be moved too, this is a smaller change
to avoid indirectly including DNA_ID.h in many places.

Apply orientation when importing USD normals.

Also added check to ensure normal and loop counts match.

Cleanup: Register node property layout callbacks in files

This commit moves the property layout callbacks for node types to their
implementation files from `drawnode.c`. This was proposed a while ago in
T75724.

**Benefits**
- Fewer files need to be changed when adding a new node.
- Makes it possible to reuse functions from the node's implementation
in the layout code.
- Except for RNA, all of the node "inputs" are in the same place.
- Code gets shorter overall, avoids the large switch statements.

**Downsides**
- Requires including two UI headers.
- Requires adding an editors dependency to the nodes folder.

This commit only changes function nodes and geometry nodes, more can be
moved later.

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

USD import: orientation bug.

Uninitialized USDMeshReader::m_isLeftHanded flag was
causing the vertex winding orientation to randomly flip,
causing incorrect shading and other issues.

Docs: Remove CLI arguments removed in recent commit

This commit follows up on rBbc94036a76b63254181788ce5814fb946f52a287
and removes the arguments from the CLI args docs.

Sculpt Expand: Force expand to work only on active connected components

Fix crash in versioning code

Sculpt Expand: Fix keymap entry for snapping

Scupt: Fix failed scene projection with ortographic views

USD import: handle primvars:normals if specified.

Per Pixar UsdGeomPointBased documentation: If 'normals' and
'primvars:normals' are both specified, the latter has precedence.

Sculpt: Fix wrong vertex normals in Scene Project

USD import: Handle empty vertex-varying normals.

If point normals are empty, now invoking BKE_mesh_calc_normals()
to avoid black renders. Since BKE_mesh_calc_normals() requires
edges to be defined, moving point normal calculation to follow
reading polys.

Fix for T84038: Improved Report Warnings

Improved contrast for Status Bar report warning messages.

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

Reviewed by Hans Goudey

EEVEE: Cleanup: Remove use of macros for the loops

EEVEE: Fix occlusion applied on SSR

Merge branch 'master' into xr-actions-D9124

Cleanup: Remove using-directive from freestyle headers

The header files in freestyle utilize the using-directive at the global
file scope. This is a bad practice as it pollutes the global name space
causing possible ambiguous reference compilation errors. In particular,
the DNA files that are included by freestyle will cause those ambiguous
reference errors when the developers adds a DNA member with a type name
that also exist in the Freestyle name space, such as Curve and possibly
others.

This patch does the minimal work needed to resolve that by moving the
using-directives from the headers into the corresponding translation
units.

Reviewed By: Brecht

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

Fix T85448: File Browser sidebar collapses when selecting a file

The logic to ensure a valid region state was too aggressive in setting the
region hiding. It would just always update it based on the operator's
`hide_props_region` option, not only when file browser was newly opened.
It's more selective now.