search

Ongzl / pe

0 stars 0 forks source link

Details function is a downgrade from AB3 #5

Open Ongzl opened 2 years ago

Ongzl commented 2 years ago

2021-11-12 (11).png

2021-11-12 (10).png

After using a command just to look at the details of the contact, I now need to use list to go back to the original list and use details again just to look at the details of another contact, this is a downgrade from AB3 where I could get all the contact info at one glance. This feature and poor UI display decision is making user take more time and test their memory. If I wanted to view details of a few contacts that I found, I have to use details and find for each contact

2021-11-12 (13).png 2021-11-12 (14).png

nus-pe-bot commented 2 years ago

Team's Response

This feature and poor UI display decision is making user take more time and test their memory. If I wanted to view details of a few contacts that I found, I have to use details and find for each contact

Firstly, it mentions that this is a UI flaw, a cosmetic detail. Secondly, it is not mentioned that we are penalized when we "downgrade" from AB3. Thirdly, I do not believe it is a "downgrade". This follows the idea of high level thinking. Does the user really want to know everyone's details when the user calls the list command? It is more likely that the user wants to have an overview. When the user wants the details, the user can then use the details command. If anything, the reporter should be suggesting that we allow details command to work on a range of indexes, which I would graciously accept. However, this is not what the tester is asking for and so we are rejecting this bug report.

I would seriously implore the teaching team to investigate matter. It seems to me that there are several insincere bug reports with inaccurate severity and they seem to be worded in a very aggressive manner intentionally.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This details feature is a nightmare to do manual testing on.

2021-11-17 (5).png

This feature of hiding contacts details and adding in an extra command to view those details feels like a deliberate sabotage to make manual testing as troublesome and difficult as possible to dissuade testers from digging deeper into the contacts list(check that edit works, check that find works etc.).

2021-11-17.png

This hiding of details and forcing the user to key in an extra command just to view the details when the details were originally shown in the contacts card in AB3 is designed to slow the user down, no matter the "idea of high level thinking" which seems that the developers are just throwing in design patterns without consideration for the users. The cons far outweigh the pros, which are entirely assumed by the developers. I would rather use my scroll wheel instead of repeatedly typing in details and list/find going back and forth just to see more details.

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: 2021-11-17 (4).png

VeryLow is reserved for flaws that are purely cosmetic and do not affect usage, this design flaw forces users to repeatedly type in commands to view details of each contact, this causes the user inconvenience for a very basic part of the app that would be used quite commonly, I believe medium would be the minimum severity for this "feature" of the app. The Dev team choosing VeryLow just seems like they are trying their luck to bargain for lower points lost and nitpicking 2 words out of an entire review and entirely out of context.