marcuschj / pe

0 stars 0 forks source link

Color coding does not make any sense for subjects #13

Open marcuschj opened 2 years ago

marcuschj commented 2 years ago

There are different colors given for the subjects, but these color codings are not consistent, defeating the purpose of color coding them.

for example, English has different color codes for tutee 1 and tutee 2.

Furthermore, the same subject has different colors for a single tutee (tutee 1).

Would prefer a standardised color code for each subject, or do away with the colors

image.png

nus-pe-bot commented 2 years ago

Team's Response

"these color codings are not consistent"

"Would prefer a standardised color code for each subject"

"or do away with the colors"

In the typical use case such as in our sample data, having 1 green coloured subject label indicates the tutee has 1 lesson. If the tutee has 2 lessons, you can easily identify the pink label as it stands out, so on and so forth. In the rare case that a tutee has 5 lessons, you can identify all of them as well. We did not feel the need to cater for more colours above 5, as not only does it become more cluttered with more colour variations, but is also highly unlikely.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It's good that you guys have thought it out and there is indexing logic behind the colours.

Firstly, I would like to point out that your UG does not have a single mention of the tag colours. Not a single use of the word 'colour' is stated in your UG. Hence all this explanation would never reach the user, and will surely be a feature flaw - this is indisputable!

To go into further details, a claim that you guys made: 'Colours are coded in sequence of ordering in the tutee's lesson list. It consistently cycles through 5 colours (green, pink, yellow, purple, and cyan) as evident in your own picture.'

It is clear that there is no cycling through the colors as seen by contact 2 (Bernice Yu)


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]