search

waynezsy / pe

0 stars 0 forks source link

Long reminders are cut off #7

Open waynezsy opened 1 year ago

waynezsy commented 1 year ago

image.png

Long reminders are quite possible (as compared to every other field), but there is no text wrapping or anything done to ensure that long reminders can be fully read from the application.

soc-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Reminder does not wrap when the reminder is long

In this case, the reminder that I would like to input has been cut short.

The severity is medium since this can affect some users.

image.png

To recreate, type

  • remind 1 r/Contact this individual regarding the insurance plan that he signed up for on the 2022-10-11.

I would have expected the text to wrap.

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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Reminders are not meant to be too descriptive as we do not want the text to wrap and cause one panel to be too long as that would result in fewer reminders being displayed.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I believe the author of the 'original' bug to which I was flagged a duplicate of provided an even more natural and convincing example of a possible user input that proves why reminders should be wrapped.

Wrapping reminders will not significantly affect the number of displayed reminders, and since reminders are sorted by date it should not be a very big issue even if less reminders can be seen without scrolling since the important and urgent ones are displayed first. Alternatively, the developers can keep the box size of each reminder the same and implement scrolling within each reminder box for longer reminders.

On the contrary, based on the current implmentation, very valid reminders that are slightly longer in length will no longer be readable. Given that the reminders feature is 1 of 2 features that greatly distinguish Friendnanacial from AB-3, I believe it is a significant feature flaw that the reminders feature is not robust for users.

Furthermore, there is no warning or mention by the developers that reminders cannot be too long within their UG/DG documentation. This seems to point towards a product design miss on the part of the developers, suggesting that this is an actual feature flaw that should not be rejected.

Another point to note is that the developers bothered wrapping person fields such as phone number, but not reminders. I believe wrapping reminders should be of higher priority than wrapping phone numbers for extremely long phone numbers, considering there is greater likelihood of needing long reminders.

:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: I believe this is a feature flaw, since the developers had no intention of wrapping the reminder text, as per their response and the fact that they wrapped certain fields for person (such as phone number) but not for reminder. There is also no warning or mention by the developers that reminders cannot be too long within their UG/DG documentation. This is hence a missing functionality that makes the reminders feature less useful for normal usage.