btbrandon / pe

0 stars 0 forks source link

Undo/Redo is unclear on what commands were executed #5

Open btbrandon opened 1 week ago

btbrandon commented 1 week ago

Commands: undo/redo

Expected: Message of the corresponding command that was undone/redone to be displayed

Actual: "Undo success!" and "Redo success!"

Explanation: It might be confusing and not assuring for users using the app, especially when multiple undos/redos are required. e.g., when the user edits the wrong field for several persons.

It might be better to display the success message of the corresponding commands that were undone or redone.

Difficulty wise, the feature should be simple to implement using the stored history of the commands.

nus-pe-script commented 1 week ago

Team's Response

It's of lower priority compared to the other planned enhancements. It is also not a straightforward implementation, as command history and undo have no association.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]