Page MenuHome

Sculpt: Sculpt Face Set gestures tools
ClosedPublic

Authored by Pablo Dobarro (pablodp606) on Aug 26 2020, 4:20 PM.
Tags
None
Tokens
"Y So Serious" token, awarded by shader."Party Time" token, awarded by tiagoffcruz."Love" token, awarded by n-pigeon."100" token, awarded by Frozen_Death_Knight."Love" token, awarded by Brandon777.

Details

Summary

This implements the sculpt gesture lasso and box operators for face
sets. They work the same way as the mask gesture operator and tools.

Diff Detail

Repository
rB Blender
Branch
sculpt-face-set-gestures (branched from master)
Build Status
Buildable 9809
Build 9809: arc lint + arc unit

Event Timeline

Pablo Dobarro (pablodp606) requested review of this revision.Aug 26 2020, 4:20 PM
Pablo Dobarro (pablodp606) retitled this revision from Sculpt: Sculpt Face Set gestures tool to Sculpt: Sculpt Face Set gestures tools.Aug 26 2020, 4:21 PM
Pablo Dobarro (pablodp606) edited the summary of this revision. (Show Details)
Sergey Sharybin (sergey) requested changes to this revision.Sep 3 2020, 3:45 PM

Similar to the comment from D8599#214204: the technical implementation of code should be brought up to 2020 standards.

Sure the code in this patch was done before that comment, so probably better to move forward with this implementation but plan to do proper refactor in the near future. For example, tomorrow ;)

P.S. The inlined comment should be addressed before commit.

source/blender/editors/sculpt_paint/paint_mask.c
549–551

Just assign, without the check.

This revision now requires changes to proceed.Sep 3 2020, 3:45 PM
Pablo Dobarro (pablodp606) marked an inline comment as done.
  • Review update
This revision is now accepted and ready to land.Sep 3 2020, 4:29 PM
This revision was automatically updated to reflect the committed changes.