rynlwhh / pe

0 stars 0 forks source link

Find command misleading part 2 #5

Open rynlwhh opened 1 week ago

rynlwhh commented 1 week ago

Similar to the previous issue raised, when I find a certain person in the contacts list panel, the details panel still shows the old details, it should reset or show the updated found details so as to not confuse users especially those not familiar with the app, and may mess up their work if they take down wrong details Screenshot 2024-11-15 at 4.30.18 PM.png Screenshot 2024-11-15 at 4.30.07 PM.png

nus-se-bot commented 3 days ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Details panel UI is not updated when a different command is entered

After using the show feature and the full details of a specified contact is displayed on the details panel, the same contact details will be displayed even after a different command is typed (eg. find alex). This can be confusing for the users as the contact details shown on the details panel may no longer be relevant after a different command is entered. It would be better to clear the details panel once a different command (i.e. not a show command) is typed to make the UI clearer.

Screenshot 2024-11-15 at 4.57.54 PM.png


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for your bug report, however this is the intended behaviour of PROperty. The client list on the left and full detail panel on the right are intended to be 2 different windows which function independently.

For example, the user might want to keep a detailed “show” of a previous user while he or she goes on to search, filter or sort the Contacts List in the left panel.

This is similar to how other popular applications such as Telegram behave. If you have a chat open and start searching for other chats/channels in the left panel, the chat you have opened will not close until you select another chat/channel to view.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.Rejected`] - [x] I disagree **Reason for disagreement:** I understand what the details panel is used for better now, but this was not stated in the UG or DG which can leave users like me confused. So I would say it could be a documentation bug instead.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** severity.VeryLow : A flaw that is purely cosmetic and does not affect usage e.g., a typo/spacing/layout/color/font issues in the docs or the UI that doesn't affect usage. Only cosmetic problems should have this label. severity.Medium : A flaw that causes occasional inconvenience to some users, but they can continue to use the product. I feel like it is not a purely cosmetic flaw and in fact as it is not specified what this details panel is for, it can cause confusion and inconvenience to users, they still can use the product which is why it is Medium severity