search

SimHanKiong / pe

0 stars 0 forks source link

Tag field different from rest #5

Open SimHanKiong opened 2 months ago

SimHanKiong commented 2 months ago

Screenshot 2024-04-19 at 4.23.34 PM.png

The difference between tag and other fields seems very unintuitive as all other fields are -FIELD but for tags is t/. Would standardising everything be better?

nus-pe-script commented 2 months ago

Team's Response

Duplicate issue

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Inconsistent prefix for the tag field.

image.png

Differences between the prefix for the tag field and all other fields: 1.All other fields used the - character to begin the field, the tag field uses the / character.

  1. The / character is placed after the field name for tag, while for the other fields the special character is before the name.
  2. All other fields are used with the full names, while t is a short hand.

These inconsistencies can be confusing for users, as they will have to remember these quirks while inputting data, which would reduce the usability of the app.

[original: nus-cs2103-AY2324S2/pe-interim#530] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Design choice to differentiate optional tags from compulsory fields when adding a student.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** This was not mentioned in the UG anywhere that this difference is intentional
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]