nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Unclear what commands `undo` undoes. #2182

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

image.png

image.png

After changing settings to 12hr form unsure if undo actually undoes the change in settings back to 24hr form.


[original: nus-cs2103-AY1920S1/pe-interim#2191]

gabrielseow commented 4 years ago

Team's Response

Accepted bug report. However, I feel that the bug has a Low severity.

Our aim for providing the undo and redo feature came from the fact that changes in +Work are hard to keep track of, as there can be changes to tasks, team members, inventory as well as the associations between these elements. For example, if a user assigned a task to a team member and wanted to undo the command, it would be very tedious to find the index of the task and the member id of the corresponding team member so that he can execute the corresponding "fire command". It would be much easier to use the undo command instead, because generally, it is hard to find out how to manually undo a command.

For the settings feature, users can visually see which setting they are on i.e. 24hr or 12hr format, Light or Dark mode and can easily 'undo' the setting change, by switching back to the setting they want. Hence, when I had written the user guide, I felt it was not important to include that detail, and could confuse the user about which commands can be undone or redone.

Duplicate status (if any):

--