benedictkhoomw / pe

0 stars 0 forks source link

Question truncates if question is long #2

Open benedictkhoomw opened 3 years ago

benedictkhoomw commented 3 years ago

How to reproduce

  1. Launch app
  2. Execute the following command(s)
  3. add q/What happens if my question happens to be veeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeery long?????????? a/e=mc^2 c/Physics p/High

Expected result

The question text wraps or provides a way to scroll to view it.

Actual result

The question text is truncated and there is no visible or documented way to see the rest of it.

image.png

Justification for severity

Medium was chosen because a long question is a reasonable input (for example, a history or law student might have to memorize long passages and might want to make them in the style: "blah blah __ blah blah"). The app is still usable but it may cause inconvenience to these users.

nus-pe-bot commented 3 years ago

Team's Response

No response provided.

The 'Original' Bug

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

Cannot view long questions in FlashCard list

In the flashcard list, you can only view the first 125 or so characters of the question. It is common for Humanities questions to be very long, and thus, you should be able to see the entirety of the question in the flash list, or at least have a scroll bar. If you intended it to be for short questions only , you should have noted it down in the UG Screenshot 2021-04-16 at 2.58.55 PM.png


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

Their Response to the 'Original' Bug

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

This is not a bug, it is a feature. We did this on purpose since this is the reason why we need view command that allows you see the completed question with answer. Also, in term of design, since a question can be extremely long that if I choose to display it fully in the list view, it can take up the whole screen, which is not visually appealing at all.

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: The behavior was not mentioned in the UG and therefore it looks like an oversight, not a feature. A user would expect it to wrap.


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: It affects a significant group of users (e.g. humanities or law students). It is not very rare. Therefore, medium.