ChocoRagdoll / pe

0 stars 0 forks source link

Background of the contact list becomes disproportionate when only a few contacts are listed #3

Open ChocoRagdoll opened 4 months ago

ChocoRagdoll commented 4 months ago

image.png

the tiles are not of the same height, but this is only a cosmetic issue.

nus-pe-bot commented 4 months ago

Team's Response

Thank you for the feedback! We have tried to solve this issue, however, due to the limitations of how JavaFX renders the provided ObservableList of persons, we were not able to adjust the height of the tiles. As we believe this is a lower-priority fix as compared to the work we’ve done so far in TaToolkit, we will be assigning this as NotInScope and will definitely incorporate it as an improvement for future iterations.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: 'due to the limitations of how JavaFX renders the provided ObservableList of persons, we were not able to adjust the height of the tiles'

I am not sure if this is a valid reason for a UI bug.

The bug should be of a higher priority that requires fixing because ALL the users opening the app with 2 or less contacts would encounter this bug.

Although being a cosmetic issue would mark this issue as a low severity one, it still hinders the user. As a user, I was unsure what the tiles meant. Are they placeholders for more contacts or something else, and why are they of different sizes? These should definitely be something the team had to think about before they finally decide on the final design of the UI.