search

gwynethguo / pe

0 stars 0 forks source link

Clickable List #9

Open gwynethguo opened 1 year ago

gwynethguo commented 1 year ago

image.png

When clicked, the patient and ward cards' colors is too dark. The user might not be able to see the contents, especially for color-blind users and users with significant eye problems.

nus-se-script commented 1 year ago

Team's Response

We use the color change to highlight the patient we selected. Selection doesn't mean anything (yet).

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I think a user might accidentally clicked the patient card without knowing what's the function of clicking the card itself. Moreover, this bug can be solved easily by changing one line: the color of the list when clicked in DarkTheme.css

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/gwynethguo/pe/main/files/562e6d2c-e84f-4df7-8bf1-469907baf7e5.png) I believe this bug should be `severity.Low` instead of `severity.VeryLow`. This is because the bug is not a cosmetic bug because it affects the usage of the tester. Because the contrast between the font and the color of the background is not that much, the tester might not be able to see the details of the patient itself, which is highly important.