search

Bandov / pe

0 stars 0 forks source link

UG rating issue #21

Open Bandov opened 2 months ago

Bandov commented 2 months ago

Does not explicitly state that values like 01 or 001 which mean the same as 1 are not allowed or should not be used Screenshot 2024-04-19 at 5.39.29 PM.png

nus-se-script commented 2 months ago

Team's Response

In the user guide, it is clearly stated that you should specify from "0" to "5" and the specific examples are also given. Since "01" is double digit, it is not within the range that we have decided on. Furthermore, for fast typists, we believe that they would not go through the trouble to add an additional "0" in front as it is inefficient.

We would also like to point out that it is also impossible to specify all abnormal formatting possibilities for the users such as "1.0", "0001" etc. Furthermore, doing so will make the user guide look unfocused and messy when it is supposed to be a concise document. Thus, we do not believe this flaw is reasonable.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Not explicitly stating that numeric input with a leading zero, like "01," are not allowed is a documentation bug due to user interface consistency, common user experience, and inclusive design principles. Users expect software to handle numerical inputs uniformly, often disregarding leading zeros, and if it does not, then it should be explicitly stated. If the software rejects such input without clear documentation which is missing in the UG, it can lead to confusion and a subpar user experience. The UG does not explicitly state anywhere that having a leading 0 in front of a digit is not allowed. They also a reinforce the users misunderstanding that they accept any form of input as long as it is a whole number value from 0 to 5 inclusive as seen in the UG and the error message. Thus, with my context and background, the user might expect leading zeros to be supported by the app. As seen below.

Screenshot 2024-04-24 at 11.04.54 PM.png

This is further supported by the fact that 01 and 1 are of equal value and thus 01 meets the requirements that is set up by the UG. See below for proof.

Screenshot 2024-04-24 at 11.12.18 PM.png

Lastly, leading zeros may be standard practice in some regions. Therefore, the application should either accept leading zeros in numeric input or explicitly state in the documentation that they are not permitted. Which neither was done.