https://issues.apache.org/bugzilla/show_bug.cgi?id=42248

--- Comment #30 from Philippe Mouawad <[email protected]> ---
Thanks for update Andrei.
First review looks good, some notes:
- you should remove import x.y.* from code, only import the classes needed
- I suggest we add an option to enable/disable history (undo/redo), we will
decide later if we enable/disable it by default
- undo/redo looks a bit confusing, as user may think he can also undo changes
in text fields or other properties of test element. We need to find a better
name although for now I am not inspired
- implementing also undo/redo on test element properties would make this
feature complete but it is more impacting

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to