nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Failure of allowing fast typing by using the application #3734

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago
  1. use cdel 1
  2. show error message, at the same time the command box is cleared
  3. need to re-type the correct command in

As stated in the documents, this application serves the fast typist to do managament. However, the users might waste time on re-typing in the command after the previous wrong input is cleared.

Screenshot 2020-11-13 at 4.33.47 PM.png


[original: nus-cs2103-AY2021S1/pe-interim#3475]

tanweijie123 commented 3 years ago

Team's Response

Thanks for your suggestion, but this feature is indeed a deliberate design choice.

image.png

We do agree that it might help in some scenarios, but we believe that the difference is negligible. A fast typist would only require less than 1 second to retype it. Moreover,

  1. If we follow your suggestion, it also means that if the user doesn't wish to continue with the deletion, they would still need to clear the command box manually.
  2. We believe that it is better to make the user fully aware of doing things that is highly destructive and irreversible (in this case the user might accidentally delete important data). This practice of making the user to re-type things is common in the software industry too. For example, deleting a repo from Github requires you to re-type the github repo name.
  3. Lastly, we believe that if the user is avid enough and is very sure that he wanted to delete all sessions with all the schedules associated to it, the user can pass in the f/ (force flag) right away. There is no restriction to doing so.

image.png

While this suggestion has its benefits, it doesn't make our current implementation a feature flaw as we believe it serves the user fairly well and having the user retype the command acts as a preventive measure for accidental force deletions.

For the reasons explained above, we would consider this issue of VERY LOW severity and we would be rejecting it as well.

Duplicate status (if any):

--