[Bf-committers] Undo for extrude function. . .what to do, what to do. . .

[Ton Roosendaal]

Hi,

Not really true. The extruder calls transform with an argument, and  
transform adds the extrude to the undo stack. Only goes wrong on ESC in  
transform.

Where to put undo pushes nicely, especially for global undo, is still a  
major task, which also could result in rewrite some of the calls.
I dont see reason for hacks, just for proper coding.

-Ton-


On 1 Oct 2004, at 13:39, joeedh wrote:

> The Extrude function currently doesn't have a BIF_undo_push.  It was  
> easy enough to add, but. . .
>
> BIF_undo_push is called twice, once for the extrusion function and  
> once for tranform();  So, as a functionality issue, should a user have  
> to press U twice to fully undo an extrusion, or should the transform  
> function perhaps be hacked to not call BIF_undo_push when in '\n'  
> mode?
>
> Heres a suggestion:  Whether or not to call BIF_undo_push can be  
> passed as part of a bitflag passed to transform().  This can be  
> implemented for the refactor of the transformation code.  By the way,  
> I really like the transformation code.
> Uhh. . .of course this isn't really my domain, but after adding my  
> individual faces extrusion function it's been really annoying to have  
> undo behave in a wierd fashion.
>
> joeedh
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at projects.blender.org
> http://projects.blender.org/mailman/listinfo/bf-committers
>
>
------------------------------------------------------------------------ 
--
Ton Roosendaal  Blender Foundation ton at blender.org  
http://www.blender.org