search

zhangchengchuan / pe

0 stars 0 forks source link

Unnecessary UI details #10

Open zhangchengchuan opened 2 years ago

zhangchengchuan commented 2 years ago

image.png

There are mentions of details such as "see person's details in the details pane". Not sure if this important but this might be an unnecessary detail.

nus-pe-script commented 2 years ago

Team's Response

It is a necessary UI detail as that is the result of show command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This is a use case. There should be no general UI details as that would hinder future development. What if in the future, CohortConnect will still display the person's details but in a different manner than in the "detail pane", such as when there is no more "detail pane" or when it is shown in a video instead?