Age | Commit message (Collapse) | Author |
|
|
|
Dragging assets onto the "Unassigned" catalog tree item will effectively
move the assets out of any catalog. Technically this means unsetting the
Catalog-ID stored in the asset metadata, or more precisely setting the
UUID to be all zeros.
|
|
|
|
Should make the code a bit more organized and help getting an overview
of the interfaces more quickly.
|
|
file`
Introduced by fc7beac8d6f4. During code review it wasn't clear why this
branch was needed, so we removed it. Now it is clear why it is needed
so we added it back and added a comment why the branch is needed.
Patch provided by @Severin.
|
|
|
|
Thanks to Patrik Olsson for spotting this.
|
|
# Conflicts:
# source/blender/editors/animation/time_scrub_ui.c
|
|
# Conflicts:
# source/blender/editors/animation/time_scrub_ui.c
|
|
This partially reverts commit e62ce9e08e919f25aad444f378947f6be932730f.
|
|
{F10761402}
With active viewport render from camera view, the camera border shows up, even when passepartout and overlays are disabled.
By moving the line-drawing code to the passepartout section, it is effectively disabled when passepartout is off.
Reviewed By: sebastian_k
Differential Revision: https://developer.blender.org/D12745
|
|
Because of legacy reasons (C string compare function returning 0 when
strings are equal), the ghash compare function is expected to return
false when hashes are equal.
|
|
Blender English should use 'American' variants, not 'British' variants.
|
|
|
|
This patch fixes a crash that was recently introduced by rB5cebcb415e76.
The reason were missing poll functions in the UI and operator.
Reviewed By: ISS
Maniphest Tasks: T91873
Differential Revision: https://developer.blender.org/D12736
|
|
|
|
|
|
|
|
|
|
The pointer was referenced before being checked.
|
|
Somehow, the file from T71329 has an empty curve profile. While that may
be a problem in itself, it's reasonable to avoid asserts or crashes when
loading or drawing such a CurveProfile. This commit just makes sure the
table always has a single vertex, and adds some checks in drawing code.
|
|
This simplifies the surface of the API for a CurveProfile.
|
|
|
|
|
|
|
|
Match eSpace_Type.
|
|
Include the short-cut text and the operator properties to make it easier
to track down the key-map item source that matched the event.
|
|
With this it is possible to select any number of assets in the Asset
Browser and drag them into catalogs. The assets will be moved to that
catalog then. However, this will only work in the "Current File" asset
library, since that is the only library that allows changing assets,
which is what's done here.
While dragging assets over the tree row, a tooltip is shown explaining
what's going to happen.
In preparation to this, the new UI tree-view API was already extended
with custom drop support, see 4ee2d9df428d.
----
Changes here to the `wmDrag` code were needed to support dragging multiple
assets. Some of it is considered temporary because a) a proper #AssetHandle
design should replace some ugly parts of this patch and b) the multi-item
support in `wmDrag` isn't that great yet. The entire API will have to be
written anyway (see D4071).
Maniphest Tasks: T91573
Differential Revision: https://developer.blender.org/D12713
Reviewed by: Sybren Stüvel
|
|
|
|
This node creates a boolean field selection of bezier spline points
that have a handle of the given type on the selected 'side' of the
contol point. This is evaluated on the point domain.
Differential Revision: https://developer.blender.org/D12559
|
|
|
|
A few items when OpenVDB is not enabled.
- Cleanup a compiler warning
- Add a node warning message
- Return an empty geometry set
|
|
A previous commit, c56526d8b68ab, which sometimes didn't drop offsets
into 'in plane' faces, as a fix to T71329, was overly aggressive.
If all the intermediate edges are in the same plane then it is fine
to just put the meeting point on the plane of the start and end edges.
|
|
This method does not have to be in a header and results in a relatively
large number of symbols to be generated (42).
|
|
This reduces the compile time, because fewer symbols have to be generated
in translation units using geometry sets.
|
|
This makes it easier to scan through the classes and simplifies
testing the compile time impact of having these methods in the header.
|
|
This is very similar to rBa812fe8ceb75fd2b.
This time the number of symbols decreases further from 1335 to 928.
Compile time of the distribute node decreases from ~2.4s to ~2.3s.
|
|
The new socket declaration api generates a surprising amount
of symbols in each translation unit where it is used. This resulted
in a measurable compile time increase.
This commit reduces the number of symbols that are generated in
each translation unit significantly. For example, in
`node_geo_distribute_points_on_faces.cc` the number of symbols
decreased from 1930 to 1335. In my tests, this results in a 5-20%
compile time speedup when this and similar files are compiled
in isolation (measured by executing the command in `compile_commands.json`).
Compiling the distribute points on faces node sped up from ~2.65s to ~2.4s.
|
|
This makes it easier to test the impact of moving them out
of the header later.
|
|
|
|
This update of the Points to Volume node allows a field to populate the
radius input of the node and removes the implicit realization of
instances.
Differential Revision: https://developer.blender.org/D12531
|
|
This simplifies the code a bit and improves compile times a bit.
|
|
This comes at a small performance cost due to an additional
memory allocation, but that is not significant currently.
|
|
This makes the classes easier to read and simplifies testing
the compile time impact of defining these methods in the header.
|
|
|
|
This makes the classes more appealing to look at and makes
it easier to see what different methods are available.
|
|
|
|
|
|
This update of the Resample Curve node allows a field to populate the
count or length input of the node depending on the current mode. The
field is evaluated on the spline domain.
Differential Revision: https://developer.blender.org/D12735
|
|
I've seen requests to remove this or complaints about this error message
quite frequently. In lots of production files it's just always going off.
It's not an actionable warning, and since "slow" is relative, it isn't
always even correct.
Differential Revision: https://developer.blender.org/D12694
|