search

ramapriyan912001 / pe

0 stars 0 forks source link

Unclear language around Undo feature alternative implementations #19

Open ramapriyan912001 opened 3 years ago

ramapriyan912001 commented 3 years ago

Screenshot 2021-11-12 at 5.37.05 PM.png

Informal language - unclear what this means - "knows how to undo/redo by itself"

nus-pe-bot commented 3 years ago

Team's Response

This explanation should be quite clear, as it's a more intuitive way of illustrating the idea that "individual command has its own way to handle the undo and redo of itself".

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: How can a command "know" how to do anything? This already shows the lack of clarity in the text.

Undo capabilities are built into each individual command class or something of the sort is appropriate, this however is not.

Either it is super informal language, or it is plain and simply unclear.

Either way, it is a clear bug.