nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

After exiting view of contacts with `view`, I am unable to scroll through the contacts list when the telegram handle is long #1352

Open nus-se-script opened 6 days ago

nus-se-script commented 6 days ago

Steps to reproduce: Step 1: Execute add name/Alice C email/johnd@example.com telegram/@johnhdsjafhdsgfdusigfxjkbcakzjsdkafguiguisdahfumdfuidsgfiysdgfsdhgkfjdshgfhdgfdshgjkdghjcvsjhkcbsdjhasdhjfsdjakfgdhsgfhadfjkshcjshasdjhjsdhafjsdkhgdhjsagfhjsbcsdhakbcsdhfgdhfbajskdhsjahdsjbchjdsfaghdsgfdshjcbdshcbsdjafhdsajgfdhskghdgfhdgfsdkahgfhhdsjfdsdsagfudisgafsdkjbcksjabcskljfasdjgfuaidsgfdusiakcsjabckjsdlbafsdlfgudsiafhsdibcjskcbsjd tag/friends tag/owesMoney github/John139 (Note that Alice C is my 13th contact.)

Step 2: Execute view n/Alice C At this point I see this:

image.png

Step 3: Execute view to exit view of contact.

image.png

This is now what I see. When I use my mouse wheel, I am unable to scroll up or down, and when I click and drag the bar on the right, I am unable to scroll up or down either. The app essentially "hangs". This issue is not resolved even after entering commands such as list. However, it returns to normal and I am able to scroll again after I execute view n/Alex Yeoh.

While the planned enhancements sections states this as a problem when tags are long, it is also a problem when telegram handles are long, as none of my contacts have long tags above.


[original: nus-cs2103-AY2425S1/pe-interim#572] [original labels: severity.Medium type.FunctionalityBug]

SwaminathanViswa commented 6 days ago

Team's Response

This issue is put as severity low since it is an extreme user input for telegram IDs to be that long.Furthermore, we also mentioned a workaround for this in our known issues. In the future iterations, we plan on improving the scrolling and user experience, making the program more user friendly as well.

image.png

Duplicate status (if any):

--