search

huixuant / pe

0 stars 0 forks source link

Unable to see complete details of event in calendar #4

Open huixuant opened 1 year ago

huixuant commented 1 year ago

Details of certain events cannot be seen fully in the popup in the calendar view.

image.png

Event selected is from sample data given (Software Engineer, GovTech).

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]

UI unable to show entire information for event in calendar if the input is too long

image.png

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

Their Response to the 'Original' Bug

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

That is not an intended feature if a user wants to look at a specific event on the calendar, the user can find the event using the find command and date of the date the event falls on.

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`] - [x] I disagree **Reason for disagreement:** While I do not disagree that the issue is a duplicate (#1511) since I believe solving one could resolve the other, I beg to differ with the team's response. In particular, I would like to point out that the consequence of these two reported issues is slightly different, and that the team did not properly address my concerns in their reply. In the case of the "original" bug (#1511), the title of the event was intentionally overflowed with "malicious" input. While the last part of the event description could not be seen from the popup, other important details like the position, company and most importantly, the start and end datetimes are still visible. Hence, the bug does not bring about as much inconvenience to the user. On the other hand, in my reported issue, I was looking at an event that is part of the sample data the team has pre-populated the app with (i.e. it is benign and legitimate). Yet, I was still unable to view the position of the internship application and the start/end datetime of the deadline **in their entirety** from the popup. This would understandably affect the user's usage of the app. In addition, I am unsure what the team meant by: > That is not an intended feature. This is because on page 17 of the User Guide, it was explicitly stated that: > Clicking on an event displayed in the calendar view shows you more details about the event. ![image.png](https://raw.githubusercontent.com/huixuant/pe/main/files/0b7d266c-6a5a-45eb-a508-415209468c59.png) Therefore, I would think that it is not too much for the user to expect to be able to see the **full, complete** details of an event/deadline from the popup. For the reasons stated above, I stand by my view that this is a valid functionality bug. However, instead of the original `severity.veryLow` that I assigned during the PE, I believe that `severity.Low` would be more suitable in this scenario on hindsight, given that the cosmetic issue is something that actually hinders the user's usage of the app.
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]