search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Display issue with the execution of command when the display window is not showing the contact list #4317

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

Description

Commands can be executed when the display is not set to the contact list (which is very understandable). However, I do feel that there is a need to switch the display back to the contact list automatically after a command is executed (except those that deals with date), so that the users can "visualize" the command result immediately and get feedback from the app.

This is also crucial in terms of error prevention. For example, when the user is deleting a client while he/she cannot "see" that client in the display list (i.e. does not know the index of that client), this may leave room for error as the user can delete a wrong client from storage by accident. This is especially disastrous since there is undo command in the app yet. Hence, I feel that the delete contact command should be executable when the display is set to the contact list.

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

senwei01 commented 1 year ago

Team's Response

Some of the commands such as editing appointment can be visualized in realtime in the calendar view. For consistency purposes, we decided to do the same for all the commands.

Duplicate status (if any):

--