Age | Commit message (Collapse) | Author |
|
Tools were relying on gizmos drawing to initialize their gizmos,
now tool gizmos are initialized immediately.
|
|
D4002 by @slumber w/ edits.
Use text colors, blending to background if they don't contrast.
|
|
Also use doxy sections & remove parens from return values.
|
|
The adjustment phase had broken assumptions after adding miters,
and sent a null problem to eigen. Fixed code to check assumptions.
|
|
|
|
|
|
We should clamp the result after multiplication.
|
|
Differential Revision: https://developer.blender.org/D4353
|
|
As the fill color alpha is set to zero, it's better keep fill option disabled.
|
|
Also capture event to avoid Move transform.
Note: Now it's using a report message. Maybe this can be removed, but without the message, the event is captured by move transform.
|
|
It's weird to draw in a layer where the stroke cannot be seen or is not saved.
|
|
Am really no a big fan of using macros for that kind of things, but
meh... C solution to do that with functions (using callbacks) is
even worse. :(
|
|
Previously, the curve self snapping would only snap to points that were
earlier in the curve structure. This was because of a simple coding
snafu of using break when meaning to use continue.
|
|
reading 2.79 .blend
Reviewers: brecht
Maniphest Tasks: T61520
Differential Revision: https://developer.blender.org/D4352
|
|
Selecting object w/ a different mode missed refreshing the screen
areas current tool.
|
|
Also skip mesh recalculation when no hide/reveal is performed.
|
|
This was caused by curves pointing to each other
creating a cyclic dependency.
While the dependency graph detects this, generating a mesh for render
recursively generates data which cashes in this case.
Add in a check to detect cyclic links.
Note, this bug exists in 2.7x too - but only crashes on render
since 2.7x didn't use 'for_render' when converting data.
|
|
These were almost exact duplicates, use one engine for drawing both
kinds of vertex color.
|
|
Black wire can be hard to see against dark blue,
especially when shading is enabled.
Use light grey, matches 2.7x.
|
|
By default wire would z-fight against the surface.
Increase the bias, also don't adjust the 'w' component
since it causes bias that depends on the view direction.
|
|
Assign to local variables when setting up passes,
avoids mix-up between similarly named struct members.
|
|
When vertex/face selection are disabled - show all wire
ignoring selection & hidden state.
|
|
Face selection was added in weight paint mode since 2.7x,
however tools hadn't been updated.
|
|
Minor change from D4342
|
|
|
|
Must use the right function to get animation updates now.
|
|
|
|
|
|
duplication.
There is no reason not to duplicate Actions too here, especially when
Materials' Actions are pretty much impossible to edit from current UI
(afaik, DopeSheet editor does not has any way to change them?).
|
|
Values outside the 0..1 range produce negative colors, so now clamp to that
range everywhere. Also fixes improper handling of hue > 2.0 in some places.
|
|
The poll was checking things related to all annotations, but now the check must be only for grease pencil objects and 3D view.
|
|
Probably fix first part of T61446.
|
|
Usual legacy/history crap in NodeTree code... Datablocks's specific
freeing code should never, ever do refcounting management, this is
handled by higher-level code from BKE_library area.
|
|
Nuke away old nodeCopyNode(), much better to use new BKE_node_copy_ex(),
which behaves as expected for the various optional flags that can be passed.
This also removes the need to handle ID refcounting in calling code
(ugly!) and allows us to remove an even uglier name from our codebase! :D
Note that this fixes three related issues actually, that bug was also
affecting copy/paste of nodes, and 'Separate with copy' operator (the
latter being actually fully wrong, since it was not refcounting
anything, not even node->id pointer...).
|
|
|
|
More or less same code was being executed twice during ID copying.
Makes no sense to add yet another switch-by-ID-type to handle
specificaly runtime data during ID copying, we already have
BKE_xxx_copy_data() functions for that.
|
|
For OIIO 2.x we must use unique_ptr. This also required updating the
guarded allocator for std::move to work. Since C++11 construct/destroy
have a default implementation that also works this case, so we just
leave it out.
|
|
|
|
Was a use-after-free during relations update.
Now we do similar dependency graph tags, but without any
extra animation update logic, which was accessing various
pointers.
Was found when looking into a file from T56635.
|
|
|
|
- Timeline theme.
- 3D view runtime variables.
- Exclude EditLatt from SDNA.
|
|
|
|
Rename 'elem_dna' to 'elem_full' to signify it has array/pointer chars.
|
|
|
|
|
|
Using ID_LIGHT or ID_ID for "Lamp" meaning, "Light" without context
being for 'not heavy'.
That rename of data-block was not really nice on that side of things :/
Related to T43295.
|
|
|
|
Using 'Regular' instead for the common meaning, this avoids having to
add some i18n context disambiguation...
|
|
The new code already loops over all instances, don't need to be counted again.
|
|
|