This patch modifies the bevel operator slightly so that it becomes modal. Moving the mouse will modify the bevel factor and E/D keys will change the behavior of the tool
On 2012/05/15 13:59:24, psy-fi wrote: Fix a few non-modal operator issues, duplicate just the bmesh ...
13 years, 1 month ago
(2012-05-15 15:04:55 UTC)
#2
On 2012/05/15 13:59:24, psy-fi wrote:
Fix a few non-modal operator issues, duplicate just the bmesh for the operator
and avoid bmesh duplication for the non-modal case
one main request - regarding storing copies of operator settings, other changes are knitpicks. http://codereview.appspot.com/6208066/diff/8001/source/blender/editors/mesh/editmesh_tools.c ...
13 years, 1 month ago
(2012-05-15 22:40:02 UTC)
#6
looks good, still some remaining knitpicks and one question. http://codereview.appspot.com/6208066/diff/3002/source/blender/editors/mesh/editmesh_tools.c File source/blender/editors/mesh/editmesh_tools.c (right): http://codereview.appspot.com/6208066/diff/3002/source/blender/editors/mesh/editmesh_tools.c#newcode4309 source/blender/editors/mesh/editmesh_tools.c:4309: ...
13 years, 1 month ago
(2012-05-16 10:44:09 UTC)
#8
13 years, 1 month ago
(2012-05-16 11:07:35 UTC)
#10
http://codereview.appspot.com/6208066/diff/3002/source/blender/editors/mesh/e...
File source/blender/editors/mesh/editmesh_tools.c (right):
http://codereview.appspot.com/6208066/diff/3002/source/blender/editors/mesh/e...
source/blender/editors/mesh/editmesh_tools.c:4653: static char str[] = "Confirm:
Enter/LClick, Cancel: (Esc/RClick), thickness: %f, depth (Ctrl to tweak): %f
(%s), Outset (O): (%s)";
On 2012/05/16 10:44:09, ideasman42 wrote:
> on topic of keymaps - this kind of string should be generated from modal
> keymaps. longer term todo.
Yes, I was thinking along the same lines too. That would be great to have.
http://codereview.appspot.com/6208066/diff/3002/source/blender/editors/mesh/e...
source/blender/editors/mesh/editmesh_tools.c:4923: RNA_def_boolean(ot->srna,
"use_relative_offset", TRUE, "Offset Relative", "Scale the offset by surrounding
geometry");
On 2012/05/16 10:44:09, ideasman42 wrote:
> why is this changed?
Because it is difficult to estimate a good absolute thickness when using screen
space mouse values. Changing to relative gives more predictable results. If the
user needs absolute values he usually will know the exact value and place it
manually.
Issue 6208066: Modal bevel operator
Created 13 years, 1 month ago by psy-fi
Modified 13 years, 1 month ago
Reviewers: bf-committers_blender.org, ideasman42
Base URL: https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Comments: 19