Rachael-Chan / pe

0 stars 0 forks source link

sort command refreshes contact list after find command #5

Open Rachael-Chan opened 2 weeks ago

Rachael-Chan commented 2 weeks ago

Bug description: After executing find, executing sort causes the contact list to refresh.

Steps to replicate: find er

Screenshot 2024-11-15 at 17.08.02.pngAfter executing find er

sort 1 n/

Screenshot 2024-11-15 at 17.09.26.pngAfter executing sort 1 n/

Suggested improvement: Maybe this could be listed in the user guide so users know that they should execute sort command before executing find command to prevent the list from refreshing.

nus-pe-script commented 2 weeks ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

From the user's point of view, it makes sense that sorting the list displays all the contacts in the sorted order so that the user can easily see the results of the sort command. If the user had used the find command, it is likely that the displayed contacts list has minimal contacts and is hence unlikely that the user would need to sort the displayed list. In the event that the user does want to apply sort on the displayed list, the auto-fill feature using the up-arrow keys can allow the user to easily re-execute the find command after sorting to show the desired results with minimal inconvenience.

Based on the above reasons, we decided to reject the bug since it is the expected and correct behaviour from the user's point of view.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]