yisiox / pe

0 stars 0 forks source link

Output box not resizable so common errors are cut off #7

Open yisiox opened 7 months ago

yisiox commented 7 months ago

Description

The output box does not seem to be resizable, leading to common errors as shown in the screenshot below being cut off. The user will likely have to use their mouse to scroll (possibly both up-down and left-right directions) to see the error in full, contradicting the typing focus of the application.

Screenshots

image.png

image.png

Reason for Severity

The errors with long feedback are expected to be common: resulting from typo by the user, command format, email format etc. The user can continue to use the application, but with possible common inconvenience.

nus-pe-bot commented 7 months ago

Team's Response

UI issues not in scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Appreciate the dev team's response. I do believe however the appropriate severity may be `Medium` still and should not be changed to `VeryLow`. I think this does not qualify as a cosmetic bug, as it does affect usage in that it slows down the user sometimes when they make mistakes. ![image.png](https://raw.githubusercontent.com/yisiox/pe/main/files/5729a6fe-70e5-4e1a-89bb-2065caa2a467.png) While this section of the course website is not addressed at error messages, I think we can reasonably extend the logic here. ![image.png](https://raw.githubusercontent.com/yisiox/pe/main/files/91e1940d-7adc-49f1-b579-ac7b14f38baf.png) For multiple commands, the error message is long enough that even at fullscreen (as per the original bug report's screenshots), it may be cut off on the right if not already cut off at the bottom. As such, I believe that it is sufficient reason to believe that it should occasionally cause inconvenience to the user, even if it can be worked around, most closely coinciding with a `Medium` severity label.