ChocoRagdoll / pe

0 stars 0 forks source link

The attendace box is too small and the behavior is inconsistent with the contact list #4

Open ChocoRagdoll opened 4 months ago

ChocoRagdoll commented 4 months ago

As a CS2100 TA I may occasionally have to record up to 10 student that are absent for a certain week. However, the current layout of the app has an attendance box that is too small for displaying all the absentees once the number of absentees exceeds 3. Hence, this may be much of an inconvenience for the user.

image.png

Currently, the left contact list's behavior is ideal, since it would automatically use a new line to show the information that would have gone out of the UI.

image.png

However, for the attendance box, the fields will just go out of the UI. In my opinion, the attendance field usually contains multiple names and is therefore longer in general, so it would be more ideal if the box's behavior could be made similar to that of the contact list.

nus-pe-bot commented 4 months ago

Team's Response

This issue was caused by the user intentionally moving the right side panel too much to the right (as shown in your images), which would cut off the content. Under normal behaviour without adjusting the side panel, the names in said example would still be visible.

As such, this can be counted as user behaviour that intentionally makes the app harder to use, which Prof said does not count as a bug. Also, this is not medium severity as the app can still be used with this, as long as you ensure that the side panel is placed well.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The moving of the panel is to show the inconsistency in behavior, which means that the title 'Attendance' would move downwards when the window is set to a smaller size, which should not be the intended behavior since the important information will all be invisible.

As see in the first picture of the bug report, 3 normal length names would already take up half the space of one attendance tile. This would be problematic when a lot of people (>10) are marked absent for one specific week, which would be rare but once occured, there is basically no way for a tutor to see the rest absentees unless he has a really huge screen.

As such, the attendance bos currently has several important issues yet to be fixed.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** The issue mentioned before is indeed rare, but once occured, the attendance box will not be able to show all the information and the fix is hard to achieve, so I would categorize this issue as of medium severity.