Blender Git Commit Log

Git Commits -> Revision b0d9e67

Revision b0d9e67 by Germano Cavalcante (master)
August 19, 2021, 13:34 (GMT)
Fix T87173: wrong Auto-Snap in animation editors

This was partially broken with {rBde9ea94fc6f}.

The `Frame Step` and `Second Step` snapping options were working as if
they were `Nearest Frame` and `Nearest Second` respectively in the
`Dope Sheet` and `NLA` editors.

In the `Graph Editor` the problem was more serious:
"Second Step: ... The keyframe itself moves along as though in snapping
were active at all, while its handles 'stay behind' until it reaches
the next second boundary, at which point the teleport handles to
'catch up'".

The snapping code for these modes was spread across the transform
mode code and `recalcData` of each data type. Therefore, create a
unified snapping code for these options so that all issues are fixed in
one place.

Differetial Revision: https://developer.blender.org/D12241

Commit Details:

Full Hash: b0d9e6797fb866e7a58876c7977c98a190070310
Parent Commit: 119d532
Lines Changed: +119, -171

9 Modified Paths:

/source/blender/editors/transform/transform_convert.c (+17, -0) (Diff)
/source/blender/editors/transform/transform_convert.h (+3, -0) (Diff)
/source/blender/editors/transform/transform_convert_action.c (+15, -0) (Diff)
/source/blender/editors/transform/transform_convert_graph.c (+6, -53) (Diff)
/source/blender/editors/transform/transform_convert_nla.c (+18, -53) (Diff)
/source/blender/editors/transform/transform_mode_timescale.c (+0, -4) (Diff)
/source/blender/editors/transform/transform_mode_timetranslate.c (+14, -19) (Diff)
/source/blender/editors/transform/transform_snap.h (+8, -7) (Diff)
/source/blender/editors/transform/transform_snap_animation.c (+38, -35) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021