nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Syntax highlighting not working as intended #1169

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

Based on the UG red is supposed to be for invalid commands while grey is for valid commands pending arguments. Based on this, the command I've typed above should be in grey as it is a valid command pending the tag information


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

SuhailLoya commented 1 week ago

Team's Response

The command is shown in red because the command is invalid, as tag cannot be an empty string. So the syntax highlighting is correct.

Some inside information: Syntax highlighting actually uses the parser to check the validity of the command. The command appears in red because the parser fails to parse the command, so by definition it is invalid.

Screenshot 2024-11-17 at 10.19.11 PM.png

Duplicate status (if any):

--