Blender Git Loki

Git Commits -> Revision 7154b42

Revision 7154b42 by Jason Wilkins (master)
May 28, 2011, 11:30 (GMT)
Fix: The radial control would fail, in sculpt mode, to set size if object-space sizing was enabled.

This was caused because a small part of sculpt's radial control code did not make it into the new version. The old code would set a new object-space size by scaling it proportional to how much the new screen-space size was changed.

The solution I implement here is to do the same scaling inside the RNA callbacks. This way, users of those properties do not have to worry about inconsistency.

I added a comment warning that brush_set_size, brush_set_unified_size, brush_unprojected_radius, and brush_set_unprojected_radius do not guarantee consistency because it is not always possible to precisely know what the new unprojected radius is in all contexts where you might set the size.

I would implement the consistency check at the lower level (in those listed functions) but at this time I think it needs to be looked at to make sure that won't cause problems. In addition, I am not sure that scaling by the ratio of change is strictly correct in all cases.

In any case, this at least fixes the immediate problem.

Commit Details:

Full Hash: 7154b424e9cd5f4b382874fc5c3cf17b18f81777
SVN Revision: 36975
Parent Commit: 8246f94
Lines Changed: +28, -0

2 Modified Paths:

/source/blender/blenkernel/intern/brush.c (+13, -0) (Diff)
/source/blender/makesrna/intern/rna_brush.c (+15, -0) (Diff)
Tehnyt: Miika HämäläinenViimeksi päivitetty: 07.11.2014 14:18MiikaH:n Sivut a.k.a. MiikaHweb | 2003-2021