search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

unmark function #1280

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

able to enter duplicate indices with no error message mark.png

[original: nus-cs2113-AY2021S2/pe-interim#1280] [original labels: severity.Medium type.FunctionalityBug]

8kdesign commented 3 years ago

Team's Response

This is the intended behavior. The correct task was unmarked, while the duplicates are automatically ignored. This was done to provide users with the best possible experience.

As for why there isn't an error message, we believe it isn't an error in the first place, unlike those with index out of bounds. Putting multiple of the same index and putting just one tells us the same thing: You want to select the item at the index. Since it is a valid index, there is no reason why we need to make it an error.

In fact, putting an error would make things worse as the user may get confused about whether the action was successful or not.

Duplicate status (if any):

Duplicate of #1281