nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Excessively long addition of field causes the fields #4100

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 months ago

Reproduction:

  1. Resize window to the smallest resolution set by the app
  2. Add / Update a contact's address / phone number to be excessively long

Expected:

Considering a scrolling list was used, I would believe that makes sense if the excessively long field were to be truncated (which it does), but the fields below it (e-mail, etc) should not be truncated as well.

Actual:

Fields below field edited and up to Date of Birth are truncated as well.

image.png

image.png


[original: nus-cs2103-AY2324S2/pe-interim#5154] [original labels: type.FunctionalityBug severity.Low]

yorklim commented 6 months ago

Team's Response

Mentioned in planned enhancements: Fix Text Truncation In Client Details - Currently, the name, tags, phone number, address, email and remark may get truncated if they are too long. In future versions, we will support the wrapping of all fields in the Client View Display.

can also be fixed by resizing window, and phone numbers / addresses are usually not unreasonably long

Problems caused by extreme user behaviors: If the problems can only happen in case of deliberate sabotage (e.g., user entered a 30-digit telephone number), in which case it should not be considered a bug. However, if such input can be entered by a user mistake, they should not cause harm e.g., such mistakes should not crash the app, corrupt the data, or make it unusable. Problems caused by integer overflows -- apply the guideline in the previous point.

Duplicate status (if any):

--