ywllowsensor / pe

0 stars 0 forks source link

Command output box is too small, cutting off output messages #5

Open ywllowsensor opened 2 weeks ago

ywllowsensor commented 2 weeks ago

Steps to reproduce: 1) Type in add_product n/Tissue Paper st/500 su/Global Produce

Actual output:

image.png

Users are required to use their mouse to scroll within the command box to view the entire error message. The product hinders users who prefer using the keyboard. This violates the project constraints which states that the product should be optimised for CLI-based users. I have tagged this as medium because error messages are important for users to be able to view and decipher what they are doing wrong. If they are not able to view the error message, then this greatly hinders them from performing their day to day tasks.

nus-se-script commented 2 weeks ago

Team's Response

No details provided by team.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The team did not provide any reasoning for this decision. Nevertheless, I disagree with this decision for the same reasons mentioned in my original bug report.

image.png

In addition, I would also like to mention that it would not take much effort to resize the window of the CommandBox by changing a few numbers in the relevant fxml files. Hence, since no work has been done specific to this bug throughout the duration of the TP, the bug does not qualify as a NotInScope bug.

image.png


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** The team also did not provide any justification for this decision. As mentioned above, the bug should be of medium severity for the reasons mentioned in my original bug report. The first thing that users will look for when a command is not executed is the error message. Cutting off the message withholds information that is potentially useful to the user. Target users are then inconvenienced by having to scroll to see the full message. I can think of a simple fix where instead of making the box bigger, the team can provide an alternative for users to use arrow keys to scroll through the message. In any case, while users can continue to use the product, it causes occasional inconvenience to the target users, which is why it classifies as a medium severity bug.