lihka1202 / pe

0 stars 0 forks source link

The UI is very hard to read through #5

Open lihka1202 opened 1 year ago

lihka1202 commented 1 year ago

Some more spacing could be added to the UI, as it is very hard to read through, especially the errors can be better spaced due to the complexity of each command:

We found 6 matching items!
> filter food -date 14/04/2023 14/04/2023
Sorry, your filter command is invalid. Filter format is as follows: 
filter {-in/-out} {-d DESCRIPTION} {-c CATEGORY} {-date DD/MM/YYYY} 
At least one of these flags must be present and in the order above.
Please refer to 'help' for more information!
nus-se-bot commented 1 year ago

Team's Response

This is an aesthetics issue. Furthermore, this error message is given in detail to let users know what is wrong with the command, and the correct format for the command as well. We think that this is much better for the user than a simple hand-waving error message that does not provide details at all.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I referred to the errors being spaced better.

Entering a new line would make quite a bit of difference in terms of readability:

For example:

We found 6 matching items!
> filter food -date 14/04/2023 14/04/2023

Sorry, your filter command is invalid. Filter format is as follows: 

filter {-in/-out} {-d DESCRIPTION} {-c CATEGORY} {-date DD/MM/YYYY} 

At least one of these flags must be present and in the order above.
Please refer to 'help' for more information!

This is easier for the users to read, especially when commands are entered one after the other.