TheDuckCow / godot-road-generator

A godot plugin for creating 3D highways and streets.
MIT License
316 stars 16 forks source link

Need to support redo/undo #54

Closed TheDuckCow closed 1 year ago

TheDuckCow commented 1 year ago

Currently, the addon does not very effectively support undo/redo operations. Meaning the user may add a roadpoint or change the number of lanes, and then if they hit control-z, instead of undoing that operation, it'll undo some other previously registered operation (without actually changing the thing intended). This applies to essentially any operation we run in the editor. To name a few that need to respect undo/redo:

See documentation here: https://docs.godotengine.org/en/stable/classes/class_undoredo.html

TheDuckCow commented 1 year ago

Bret tried working on this for the addition of adding a road point, the undo point wasn't working well and had some side consequences which we want to investigate.

TheDuckCow commented 1 year ago

All but two are now addressed. Officially closing now, remaining two going into backlog.

Edit: Posted the extracted out commands here, including some additional searching that I wish I had done sometime earlier: https://github.com/TheDuckCow/godot-road-generator/issues/84