diff options
author | Pablo Dobarro <pablodp606@gmail.com> | 2021-04-18 20:41:09 +0300 |
---|---|---|
committer | Pablo Dobarro <pablodp606@gmail.com> | 2021-04-27 22:40:54 +0300 |
commit | a4191c2d184705e18177b10f1f91eea3243a475c (patch) | |
tree | 5439390a8625a4933078eb4834a29e635ad47c7b /source/blender/editors/sculpt_paint | |
parent | cc72dd73760ab41442e4ce7c81bea8697207d27d (diff) |
Fix T87474: Face Set visibility updates failing with single face loop
Face set visibility is already flushed to all mesh elements in the right
way in SCULPT_visibility_sync_all_face_sets_to_vertices, calling
BKE_mesh_flush_hidden_from_verts is legacy code from the previous
visibility system that was causing the vertex visibility to take
priority over face sets. When hidding a single loop, all vertices of
those faces are still visibile, so this line was tagging all loop faces
visible. This was leaving mesh/face set visibiltiy in an unconsistent
state.
Reviewed By: JacquesLucke
Maniphest Tasks: T87474
Differential Revision: https://developer.blender.org/D11008
Diffstat (limited to 'source/blender/editors/sculpt_paint')
-rw-r--r-- | source/blender/editors/sculpt_paint/sculpt_face_set.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/source/blender/editors/sculpt_paint/sculpt_face_set.c b/source/blender/editors/sculpt_paint/sculpt_face_set.c index 17c4beab086..bdbdb75732a 100644 --- a/source/blender/editors/sculpt_paint/sculpt_face_set.c +++ b/source/blender/editors/sculpt_paint/sculpt_face_set.c @@ -949,10 +949,6 @@ static int sculpt_face_sets_change_visibility_exec(bContext *C, wmOperator *op) MEM_SAFE_FREE(nodes); - if (BKE_pbvh_type(pbvh) == PBVH_FACES) { - BKE_mesh_flush_hidden_from_verts(ob->data); - } - SCULPT_tag_update_overlays(C); return OPERATOR_FINISHED; |