tengcharmaine / pe

0 stars 0 forks source link

Truncation of information in the app #12

Open tengcharmaine opened 5 months ago

tengcharmaine commented 5 months ago

Screenshot 2024-04-19 at 4.56.57 PM.png

I tried adding long phone numbers into the app which was possible but it got truncated at the end. While i understand that it is not possible for someone with such a long phone number, your planned enhancements mentioned about limiting the max to 500 characters for each field. This might cause users to not be able to see the all the information they need if it gets truncated. Might be better to wrap the text if it gets too long instead.

nus-pe-bot commented 5 months ago

Team's Response

Thank you for pointing this out. However, we relabeled this to have a very low severity as it is purely cosmetic and it is stated on the course website that "not seeing the last part of a very long name might not hinder the user", thus justifying a very low severity. In our case, you can clearly see far beyond the first few characters. The same can be said for every other field as you can clearly see the beyond the first field characters. Furthermore, we consider this to be a borderline case of deliberate sabotage as no valid phone number is ever that long. Yet, we acknowledge that we would cap 500 characters for each field in our planned enhancement. As such, we are not rejecting this bug entirely.

Screenshot 2024-04-21 at 1.13.03 AM.png Screenshot 2024-04-21 at 1.11.30 AM.png

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]