search

albertarielw / pe

0 stars 0 forks source link

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

Open albertarielw opened 2 years ago

albertarielw commented 2 years 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

nus-pe-script commented 2 years 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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Then, this constitutes a documentation bug.

It is not immediately clear to me or users that you cannot undo an "undo" since your user guide clearly states that undo can undo any commands other than help.

Mismatch between expected behavior based on UG and actual behavior is considered a bug as per CS2103T Website

You can check this link here to clarify https://nus-cs2103-ay2223s1.github.io/website/admin/tp-pe.html#pe-overview-2

image.png