search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Unclear sorting condition #400

Open nus-se-script opened 2 weeks ago

nus-se-script commented 2 weeks ago

image.png After sorting:s low, it will return the result as shown above.

image.png The sorted contacts have unclear order other than the priority. Should be made clearer in the UG or implement some order such as by phone number, name, etc.

Could become problematic when scrolling to find a user.

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

O-Darren commented 2 weeks ago

Team's Response

Hi,

Thank you for reporting this.

However, we believe that if someone was looking for a specific contact, they should be doing so via the find command. It is unlikely for the user to need to know how contacts are sorted within each priority bracket.

For the same reason, we believe that the severity should also be low, as it is unlikely to cause any issues for those using the application.

We also believe this would be a documentation bug, as it is behaviour not being documented in the UG, rather than a feature flaw. It is currently implemented already, and is sorted by the ASCII characters of the email.

Duplicate status (if any):

--