lululwtv / pe

0 stars 0 forks source link

Light Theme text unreadable #9

Open lululwtv opened 8 months ago

lululwtv commented 8 months ago

image.png

In light theme, the status text is quite hard to read. Perhaps we can consider using a different font colour for this?

nus-se-bot commented 8 months ago

Team's Response

The colour of the status badge is indicative of the status (green for completed and red for missed). Additionally the status text is not impossible to read.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

I believe this is still a cosmetic bug nonetheless. Since the text exists, it should at least be easy to read with contrasting colours. Moreover, the text gets harder to read when displayed on a small screen. On my laptop (a 13" screen), and when the screen is not cropped to the window (as it is in the screenshot), the text gets even harder to read.

While the colour of the box itself is indicative of the status, I believe that this does not make the text colour issue a non-bug. I also understand that the colour itself reflects the status, hence I marked this as a very low severity since this does not cause any inconvenience at all. I just believe that from a software engineering perspective, since this is a feature that has been implemented, it should at least be done so properly and the text should be made readable.