search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

The confirm message of delete does not add much value to safe deleting. #5559

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month ago

The current confirm warning provided has no information about the person. Currently, just an index number, which is not enough to check whether the user is deleting the right person. If it tries to confirm, the message should at least try to provide some information(name) about the person to check it. It is better if any type of information is provided to confirm deletion. There is no undo function, so this is quite risky. image.png

[original: nus-cs2103-AY2324S2/pe-interim#5536] [original labels: severity.Medium type.FeatureFlaw]

wang-xinrong commented 4 weeks ago

Team's Response

Although the suggestion is valid and more information can indeed be provided for the confirm popup, the current implementation does not affect the user. To view the data he wishes to delete, he can simply cancel the transaction, scroll the UI to access the indexed contact before he continues running the delete INDEX command again.

However, we recognise that such a feature suggestion can improve user experience and are willing to accept it as a cosmetic issue to be fixed. Thus the very low severity.

Duplicate status (if any):

--