chanyijuan / pe

0 stars 0 forks source link

UI of procedure/client panel could be improved #10

Open chanyijuan opened 2 years ago

chanyijuan commented 2 years ago

Unclear which clients' procedures are being shown on the Procedure Panel at any time. When commands such as editProc are run on a different client that the current procedures shown, the tab switches automatically but no message is shown.

I got quite confused many times while testing, as to which clients' procedures are showing. Would be great if there was some sort of UI indication such as the client is coloured differently/spotlighted in the Client Panel.

nus-pe-bot commented 2 years ago

Team's Response

This is an acceptable feature flaw. However, we have downgraded to low for following reasons:

image.png

  1. The behavior of switching to different Procedure is intended, as stated in the UG. This is actually requested by the previous testers so that they wish to see whether the new Procedure has been added, though it might belong to a Client that the users were not viewing at the moment.

  2. The case in which the user browse through the application in those ways mentioned are rare. The most common use case for these technicians would be after they have arrived to a Client's business site, then they open our app and check what Procedures have been done to that Client. It is even rarer that they add a Procedure to a Client, and at the same time edit a Procedure of another Client, ... and so on. (In other words, not much room to cause confusion under normal circumstances)

  3. listProc comes as a good solution. Whenever they are confused, they can simply type in this command to retrieve the Procedures of the Client they want to know more about. It is simple to type (especially for fast typists) and the result gets executed quickly.

But having more visual indication of the Procedure list sounds like a good idea and could be incorporated at the next iteration. Thank you for your suggestion.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]