nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Undo unable to undo some commands the UG claims to be able to #3793

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Steps to reproduce: clear -p undo undo

Expected: undo will undo the last undo (based on UG description of undo, it claims the app is able to do this).

Actual: no previous command error message

Screenshot:

image.png


[original: nus-cs2103-AY2223S1/pe-interim#3620] [original labels: type.FunctionalityBug severity.Low]

zacchaeuschok commented 1 year ago

Team's Response

Rejected because it is a common understanding that undo should not undo undo commands.

Our UG also does not explicitly claim that undoing an undo command is allowed.

Should such behaviour be allowed, the undo feature will essentially be useless because only an infinite loop of undo commands can be entered i.e. previous commands cannot be reversed.

Duplicate status (if any):

--