Blender Git Commits

Fix T43848: Wrong direction recalculating normals

Face islands furthest faces could face towards the center point when connected to sharp edges.

Now check the furthest edge of the furthest face, to test for face winding.

BMesh: calc normals, use area weighted center

Prevents many small faces skewing center calculation.

cleanup: float/double promotion, redundant casts

Bugfix T43851: Creating a new action after stashing crashes blender

Use the old double-step collision method only for cloth.

This method does not work for hair anyway. Even though hair collision
needs work at this point, it's still better than nothing.

Fix T43406: Bring back cloth collisions.

This was disabled during the course of hair dynamics work. The cloth
collision solution is based on a secondary velocity-only solver step.

While this approach is usable in general, the collision response
calculation still does not work well for hair meshes. Better contact
point generation is needed here (Bullet) and preferably an improved
solver for unilateral constraints.

I18n: Do not skip Polish anymore, translation work has started on it.

NLA Drawing Tweaks

* Strips in muted tracks are also drawn with dotted borders now to make it clearer
that they are not contributing.

TODO:
* Perhaps dotted should be used for protected, and opacity for muted instead?

Tweaks for NLA and newly added Action Stash Tracks

* Do not make Action Stash Tracks or their strips active/selected
* Lock the track to prevent accidental editing/adding of other strips

* Prevent strips from being added into locked tracks by the pushdown operator.
This is mainly to prevent pushdown actions from getting into the stash tracks.

Action Editor: "New Action" operator now stashes old actions, and is used by default again

This commit modifies the "New Action" operator to always stash the old action
before it creates a new one. As a result, the old active action will now have
a proper user of sorts after the new one is created, preventing previously
created actions from being lost.

Now that the New operator does this, it can be used for the Action Editor header AND
NLA Editor (Animation Data Panel -> Active Action) again. The "stash and create"
operator is somewhat redundant at this point as a result.

Action Editor WIP: Adding new actions in Action Editor now uses the "stash and create new" operator

In constrast to the old "new" operator, this operator will stash the existing action
in the stack to prevent it from being lost. This situation isn't totally ideal yet,
since the NLA Editor still calls the old method.

Action Stashing: Don't allow an action to get stashed more than once

Reshuffle code

Code Cleanup: Deduplicate logic for getting AnimData block associated with Action Editor modes

Code cleanup: Split off the creation and setting logic for new actions into different functions

This is also just as prep for the next step

UI Tweak: Display labels for Push Down and Stash

I'm still not sure which version is better, but I suspect that with the labels,
this might help users figure this out more than if they were just unlabelled
icon buttons...

Code Cleanup: Separate out the logic for stashing actions into a separate function

Action Editor: Stash Action Operator

This operator (the snowflake icon, beside the pushdown button on the Action Editor
header) adds the currently active action to the NLA stack in a muted track, then
creates + loads a new action ready to be populated with new keyframes.
Since the NLA is being used to hang on to all the actions here, no actions are
getting lost.

Usage Notes (there will be some additional tweaks to make this nicer):
* To preview different actions that have been "stashed", simply click the "Solo"
toggle for the track containing the action in question. Playing back the NLA will
now show the stashed track
* To edit a previously stashed action - simply enter tweakmode on it in the NLA
while the "Solo" toggle is enabled.

Todo:
* Add some more operators here to polish up the Action <-> NLA bridge to make the
layered and stash workflows smoother. Examples include some tools to easily
switch between the different actions layers in the stack, as well as making it
easier to get out of tweakmode (and sync up the action lengths)

* Review and cleanup the behaviour of the "new" operator here to avoid the old
problems that users were running into

* After the next release - Implement the full Action Libraries functionality, with
ways to bridge the stashed strips over to a full-blown library.

NLA: Auto-Blends option is no longer enabled on all newly added strips

This may be put back again later, but currently, this is a bit too trigger-happy
to have it on. It tends to mess up results more than it helps.

Fix for NLA Solo/Mute behaviour

The Solo and Mute functionality for the NLA system should really be mutually
exclusive features. They both affect whether a given track applies or not.
The only difference is that the Solo option mutes all the others, while the Mute
only does this on a per track basis.

Before this fix, muting a strip and then making it solo meant that the solo'd
track would not play at all, which isn't really what we want.