search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Very long names don't wrap around in the UI. #5679

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

As seen in the image below, very long names are not fully displayed in the GUI. Perhaps you can consider making the text wrap around. An alternative or addition could be to limit the number of characters allowed in the name field. To recreate, just use the add command and add a very long name after the n/ prefix.

image.png

[original: nus-cs2103-AY2122S1/pe-interim#5653] [original labels: severity.Low type.FeatureFlaw]

yongxiangng commented 2 years ago

Team's Response

Firstly it is a cosmetic issue, severity should be veryLow.

Making the text wrap around is an ok suggestion, but tester can always just input a longer string and cause it to do some weird UI bug. limiting the number of characters allowed in the name field will only decrease the utility for the user. I believe if the tester really wanted to see the entire name, he could have expanded the application window to the max, and see everything.

Additionally, most people don't have such long names. Even if there are people with such long names (maybe some), our UI is already showing a good amount of the name, so it is not the case where by the user is unable to differentiate between the 2 persons. This does not decrease the utility of the application and this bug report is somewhat unreasonable with the length of the name, so we're going to reject it.

Duplicate status (if any):

--