search

Inno97 / pe

0 stars 0 forks source link

Padding whitespace to commands. #2

Open Inno97 opened 4 years ago

Inno97 commented 4 years ago

Padding whitespaces to the front of commands will add them to the auto-suggest feature, if the command works.

I tried this out with the help command:

" help"

I don't think this is an intended feature.

image.png

nus-pe-bot commented 4 years ago

Team's Response

The auto suggest will record any inputs that results in a working command. Whitespace in front of a command is acceptable as our parser knows how to ignore it, so any commands with white space in front of them will still work and as a result be recorded.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]

Inno97 commented 4 years ago

Sure, though it's quite ugly when it happens. You could trim the string before putting it in the auto suggest, makes it look nicer in the off chance that this actually happens.