nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Sorting resets the filtered list of patients #1567

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

Steps to reproduce:

  1. Use findt to search for a tag that more than 2 people have
  2. Use sort to sort the results according to name

Expected: The command sorts the results of my find operation

Actual: It resets my search results and sorts the entire list


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

ZhiWei1010 commented 6 months ago

Team's Response

Hello, thanks for pointing that out, however. in the UG it is stated that the sort command will sort all existing patients in patient list meaning that all existing patients in PatientSync will be sorted instead of the currently displayed patient list. Adding the all existing patients would be intuitive enough for users to infer that patient list is referring to the entire patient list in PatientSync instead of what was displayed.

A more specific phrasing would be to rephrase the UG would be to replace the word patient list to PatientSync to avoid confusion. Thus changing this to a documentation bug. Furthermore, find result usually will not show many patients thus will be uncommon for users to use sort command after find command, which is why changing severity to low.

image.png

Duplicate status (if any):

--