search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Editable output response box #1575

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Selecting/Putting the cursor on the cli output response box and clicking and pressing ctrl+backspace or ctrl + delete will actually edit the response text. Proof to help explain below.

Before:

image.png

After:

image.png

Why is this an issue? The output box is in close proximity with the command input box. Ctrl + Backspace or ctrl + delete is a legit keyboard shortcut to delete word by word. Hence, users may inevitably misclick considering their close proximity and use ctrl + backspace to delete the input box word by word, only to see the output box getting deleted word by word. This can be annoying to users as they are probably using the output box to refer to command format when editing a wrong input command format for example.

[original: nus-cs2103-AY2425S1/pe-interim#2168] [original labels: severity.Low type.FunctionalityBug]

Wxy2003-xy commented 1 week ago

Team's Response

As mentioned on the course website,

Issues with the output shown in the terminal: While the terminal output is not critical to the functioning of the product, it is expected to 'behave' in a reasonably presentable manner e.g., avoid showing misleading or alarming information to the user who happens to glance at the terminal output. Therefore, issues related to the terminal can be classified at comparatively lower severities and might qualify to be NotInScope.

Very good effort for spotting this, but I believe it is not in the scope to handle all the command shortcuts. I could possible alt + F4 your programme and close it and call it a feature flaw because you didn't handle a shortcut.

Duplicate status (if any):

--