Sculpt: Invert Expand behavior for masking consistency
This patch makes sure that each of the expand keymap entries will use consistent "invert" and "use_mask_preserve" properties. Based on previous discussions we decided to flip the default Mask Expand behavior. This has multiple benefited: - The mask creation is more consistent with other masking tools (Always add to existing mask. Mask selected areas) - It's easier to use expanding for masking face sets (Snapping with `Ctrl`) or building a mask from repeated operations - It's less likely to mask certain areas unintentionally (Loose mesh islands) - If the current behavior is desired for an expand operation the user can use `E` & `F` in the modal keymap (Which is less often the case). If we want to revisit the original design of inverted masking again in the future we should do this via {T97903}. Reviewed By: Joseph Eagar Differential Revision https://developer.blender.org/D16434 Ref D16434
This commit is contained in:
parent
a9cb66b856
commit
dce7917717
Notes:
blender-bot
2023-05-23 10:40:34 +02:00
Referenced by issue #108164, Sculpt: Expand mask by topology doesn't invert the mask
|
@ -5049,20 +5049,31 @@ def km_sculpt(params):
|
|||
{"properties": [("mode", 'SMOOTH')]}),
|
||||
# Expand
|
||||
("sculpt.expand", {"type": 'A', "value": 'PRESS', "shift": True},
|
||||
{"properties": [("target", "MASK"), ("falloff_type", "GEODESIC"), ("invert", True)]}),
|
||||
{"properties": [
|
||||
("target", "MASK"),
|
||||
("falloff_type", "GEODESIC"),
|
||||
("invert", True),
|
||||
("use_auto_mask", True),
|
||||
("use_mask_preserve" , True)]}),
|
||||
("sculpt.expand", {"type": 'A', "value": 'PRESS', "shift": True, "alt": True},
|
||||
{"properties": [("target", "MASK"), ("falloff_type", "NORMALS"), ("invert", False)]}),
|
||||
{"properties": [
|
||||
("target", "MASK"),
|
||||
("falloff_type", "NORMALS"),
|
||||
("invert", False),
|
||||
("use_mask_preserve" , True)]}),
|
||||
("sculpt.expand", {"type": 'W', "value": 'PRESS', "shift": True},
|
||||
{"properties": [
|
||||
("target", "FACE_SETS"),
|
||||
("falloff_type", "GEODESIC"),
|
||||
("invert", False),
|
||||
("use_mask_preserve" , False),
|
||||
("use_modify_active", False)]}),
|
||||
("sculpt.expand", {"type": 'W', "value": 'PRESS', "shift": True, "alt": True},
|
||||
{"properties": [
|
||||
("target", "FACE_SETS"),
|
||||
("falloff_type", "BOUNDARY_FACE_SET"),
|
||||
("invert", False),
|
||||
("use_mask_preserve" , False),
|
||||
("use_modify_active", True),
|
||||
]}),
|
||||
# Partial Visibility Show/hide
|
||||
|
|
Loading…
Reference in New Issue