From 87db3423c98ea5d11a6bdbef6658b67970023a4e Mon Sep 17 00:00:00 2001 From: Sergey Sharybin Date: Wed, 20 Jan 2021 12:52:12 +0100 Subject: Fix Lock to Selection does not work in mask mode A regression since 2.80: need to use evaluated mask to calculate its bounds. Non-evaluated mask does not contain state for the current frame, so iterating over control points of the original mask gives points state from the time they were edited last (aka, not affected by the animation). --- source/blender/editors/mask/mask_query.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/source/blender/editors/mask/mask_query.c b/source/blender/editors/mask/mask_query.c index cf5997d8a18..cfd57ca3477 100644 --- a/source/blender/editors/mask/mask_query.c +++ b/source/blender/editors/mask/mask_query.c @@ -606,7 +606,14 @@ void ED_mask_point_pos__reverse( bool ED_mask_selected_minmax(const bContext *C, float min[2], float max[2]) { + Depsgraph *depsgraph = CTX_data_depsgraph_pointer(C); Mask *mask = CTX_data_edit_mask(C); + + /* Use evaluated mask to take animation into account. + * The animation of splies is not "flushed" back to original, so need to explicitly + * sue evaluated datablock here. */ + Mask *mask_eval = (Mask *)DEG_get_evaluated_id(depsgraph, &mask->id); + bool ok = false; if (mask == NULL) { @@ -614,7 +621,7 @@ bool ED_mask_selected_minmax(const bContext *C, float min[2], float max[2]) } INIT_MINMAX2(min, max); - for (MaskLayer *mask_layer = mask->masklayers.first; mask_layer != NULL; + for (MaskLayer *mask_layer = mask_eval->masklayers.first; mask_layer != NULL; mask_layer = mask_layer->next) { if (mask_layer->restrictflag & (MASK_RESTRICT_VIEW | MASK_RESTRICT_SELECT)) { continue; -- cgit v1.2.3