Page MenuHome
Differential D5846

Sculpt: Separate option for dynamic mesh preview
Needs RevisionPublic
Actions

Authored by Pablo Dobarro (pablodp606) on Sep 18 2019, 6:48 PM.

Details

Reviewers
Jeroen Bakker (jbakker)
Summary

This allows enabling dynamic mesh preview without changing the behavior
of the brush.

Diff Detail

Repository
rB Blender
Branch
dynamic-mesh-preview-option (branched from master)
Build Status
Buildable 5020
Build 5020: arc lint + arc unit

Event Timeline

Pablo Dobarro (pablodp606) created this revision.Sep 18 2019, 6:48 PM
Harbormaster completed remote builds in B5020: Diff 18340.Sep 18 2019, 6:48 PM
Pablo Dobarro (pablodp606) added a reviewer: Jeroen Bakker (jbakker).Sep 18 2019, 6:48 PM
Jeroen Bakker (jbakker) requested changes to this revision.Sep 19 2019, 11:45 AM

Don't think this is a brush option anymore. Seems more like an overlay option / tool setting. Check with UI/UX team where to place such an option.

This revision now requires changes to proceed.Sep 19 2019, 11:45 AM
Pablo Dobarro (pablodp606) added a subscriber: William Reynish (billreynish).Sep 20 2019, 1:34 AM

I talked to @William Reynish (billreynish). It should be fine to have the property in the UI like in this patch because it only affects the grab brush. It could also go to the display panel, but we want to remove it to simplify the toolbar.

Revision Contents
Changeset List

PathSize
release/
scripts/
startup/
bl_ui/
2 lines
source/
blender/
blenloader/
intern/
4 lines
editors/
sculpt_paint/
4 lines
makesdna/
2 lines
makesrna/
intern/
8 lines

Diff 18340

View Options

release/scripts/startup/bl_ui/space_view3d_toolbar.py

Loading...
View Options

source/blender/blenloader/intern/versioning_270.c

Loading...
View Options

source/blender/editors/sculpt_paint/paint_cursor.c

Loading...
View Options

source/blender/makesdna/DNA_brush_types.h

Loading...
View Options

source/blender/makesrna/intern/rna_brush.c

Loading...