tingkai-mai / pe

0 stars 0 forks source link

Missing Tag Explanation #15

Open tingkai-mai opened 1 year ago

tingkai-mai commented 1 year ago

Issue:

The display / storage behavior of a tag is not explicitly stated in the User Guide, after I input a tag. For example, if I executed radd s/1 d/1/1/2000 c/test r/1 t/TEST21, the Tag is converted from "TEST21" to "test21".

However, under:

image.png

I understood this as "I must add a Tag nonempty alphanumeric token without spaces, case insensitive", so I can do t/TEST21, but I was confused: Why was it then converted to lowercase, and displayed in FoodWhere as lowercase? (Refer to Review 7)

image.png

This is an issue as I would expect capitalization, but this was not stated in the User Guide.

To Reproduce:

N/A

Expected Output:

N/A

Actual Output:

N/A

nus-pe-script commented 1 year ago

Team's Response

Severity should be low since there is an explanation of how tags are stored in FoodWhere.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: That is NOT a proper explanation of how Tags work.

That is just the description of how a Tag is STORED, but does NOT describe its behaviour when displayed to FoodWhere's UI.

For example, if I were the user, I do not care about how the Tag is STORED. That's up to FoodWhere.

What I DO care about, is how the Tag would look like when it's displayed to FoodWhere. What if I really needed the capitalization to separate out e.g. "JAR" vs "jar"? Then because FoodWhere did not say that "all your tags will be converted to lowercase and displayed as lowercase only", I would be very confused as to why it's always lowercase.

As such, because the documentation does not mention this strange behaviour, I think this issue is valid.