search

weibinlim / pe

0 stars 0 forks source link

Error message for deleting repeated invalid indices is confusing #7

Open weibinlim opened 2 weeks ago

weibinlim commented 2 weeks ago

Description: The error message for deleting a valid index followed by an invalid index is confusing, despite it being mentioned in the UG. Since it only displays the error message, if there are contacts that are not displayed and I would like to delete with repeated indices contacts that are out of the display pane, I would not know if my command was executed successfully and to what extent. Users would have to manually expand the window to view the contacts as the scrollbar doesn't work. If the number of contact extend beyond what the users window can display, then it would be pretty hard to determine if the command was executed successfully or not.

Steps to reproduce: Add until there are 10 contacts in the list. Then run the "delete 9 9 9".

Actual: Error message for delete shows and the last 2 contacts are deleted

image.png

image.png

image.png

nus-pe-script commented 1 week ago

Team's Response

Thank you for the bug report!

The behavior of the delete command with repeated indices is already specified in our user guide as shown in the image below and the scrollbar issue is duplicated.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]