From 60746363875a67ae15278d8e4629f898d9562c47 Mon Sep 17 00:00:00 2001 From: Wayde Moss Date: Mon, 14 Dec 2020 17:23:04 -0500 Subject: NLA: Action Strip Defaults Sync Length On See T82309#1055564 {T63675} and their duplicates for how Default-off can cause confusion. This is just for convenience since it allows animators to keyframe outside of the strip's bounds by default. This was likely off by default before since Syncing Length would undesirably shift the whole animation after leaving tweak mode (fixed by {D7602}) and the animator wasn't able to keyframe outside the strip bounds anyways (fixed by {D7533}). Now it's better if the flag was on by default. While the animator is still roughly developing their animation NLA-wise they won't have to worry about strip bound keying failures. When they are more certain of the strip bounds, they can disable the flag to prevent affecting the rest of the NLA system. Reviewed By: sybren Differential Revision: https://developer.blender.org/D9661 --- source/blender/blenkernel/intern/nla.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'source/blender') diff --git a/source/blender/blenkernel/intern/nla.c b/source/blender/blenkernel/intern/nla.c index ebd9317fcf1..de0d67d386b 100644 --- a/source/blender/blenkernel/intern/nla.c +++ b/source/blender/blenkernel/intern/nla.c @@ -334,11 +334,8 @@ NlaStrip *BKE_nlastrip_new(bAction *act) /* generic settings * - selected flag to highlight this to the user * - (XXX) disabled Auto-Blends, as this was often causing some unwanted effects - * - (XXX) synchronization of strip-length in accordance with changes to action-length - * is not done though, since this should only really happens in editmode for strips now - * though this decision is still subject to further review... */ - strip->flag = NLASTRIP_FLAG_SELECT; + strip->flag = NLASTRIP_FLAG_SELECT | NLASTRIP_FLAG_SYNC_LENGTH; /* assign the action reference */ strip->act = act; -- cgit v1.2.3