search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent output #5828

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to reproduce
show 3
show -n

Expected result
One consistent output display.

Actual result
Two different output displays; show 3 keeps the list of students on the side, show -n How Kai Xin only shows How Kai Xin's display

Screenshots

Screen Shot 2021-11-12 at 4.42.28 PM.png Screen Shot 2021-11-12 at 4.43.08 PM.png

[original: nus-cs2103-AY2122S1/pe-interim#5800] [original labels: severity.Low type.FunctionalityBug]

jonas-chow commented 2 years ago

Team's Response

When showing by index, it is likely that the user already has the student in interest showing on his screen (since he would need to scroll to the student to view his index). As such, we do not filter the list.

When showing by name, we filter the list to ensure that the student's details show up in the student list on the left, so the user can take a closer look at the student's details.

Duplicate status (if any):

--