JoelChanZhiYang / pe

0 stars 0 forks source link

UI overflow for occupants of room #7

Open JoelChanZhiYang opened 2 years ago

JoelChanZhiYang commented 2 years ago

image.png

While normally this is likely to be a verylow severity bug, this UI bug is likely to cause major inconvenience as the person cannot see who is occupying the room. The overflow occurs with only 3 guests checked in

Therefore, I placed this to be a low severity bug as it causes inconvenience

nus-pe-bot commented 2 years ago

Team's Response

This issue arises from the tester's deliberate minimising of the width to the very minimum allowed. We actually wanted to set a minimum where this issue does not occur, something which we originally accounted for as we have implemented the algorithm to wrap at a certain width. But we were also mindful of a very small number of users with screensizes hence went against setting a minimum width.

image.png

Furthermore, it is mentioned in UG that for users with small screen size, they should use the application in full screen to enjoy the application's full suite of features.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: image.png

According to the textbook, it is entirely within scope to ensure that the application is usable in certain reasonable screen sizes.

During the practical, I did not intentionally minimize the width to its minimum. In fact, I used the default window size when the application started up.

The bug shows up when using the setting recommended by the textbook

image.png

Therefore, I believe it is entirely within scope for an application to be usable at the window widths I used and is not out of scope


:question: Issue type

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

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: This bug happens very often, as long as there are more than 3 guests in a room. Therefore, it cannot be considered a rare occurrence.

Furthermore, this bug has serious consequences with no workarounds. When this bug occurs, you would not be able to see who is occupying a room, defeating the entire purpose of the application. There is also no workarounds like horizontal scrollbars or text-wrap that would allow you to view the guests in a room. I feel that this bug could be justified as a severity.Medium as it causes the application to become unusable.

Lastly, I feel that including the advisory inside the UG does not nullify the severity of the bug. As you can see in the screenshot below, if a user has a screen resolution of 1280 x 720 and 150% zoom. the bug still occurs even in fullscreen mode

image.png