search

T-Wan-Lin / pe

0 stars 0 forks source link

Venue list UI issues #8

Open T-Wan-Lin opened 1 year ago

T-Wan-Lin commented 1 year ago

Screenshot 2023-04-14 at 5.02.41 PM.png

It would help if the venue size was in the UI, like at the end, you can have Venue Size: 100, instead of a raw number 100.

nus-pe-bot commented 1 year ago

Team's Response

Having the size displayed beside the venue is purely for cosmetic purposes, hence this bug does not affect the feature usage, but we agree with the suggestion to make it clearer in the UG.

The 'Original' Bug

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

Insufficient information for size of venue

Screenshot 2023-04-14 at 4.52.28 PM.png It is quite hard to see that the size of the venue is beside the venue, it took me a long time to see that and it would be better if "size : 50 " is given in the help command or in the venues list . It is also not specified that the size would be beside the venue name in the "List venues" part of the UG.

[original: nus-cs2113-AY2223S2/pe-interim#2845] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

The filter venue would just show the venues that are big enough (as specified by the user), thus having the size displayed beside the venue is purely for cosmetic purposes, hence this bug does not affect the feature usage, but we agree with the suggestion to make it clearer in the UG.

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.NotInScope`] - [x] I disagree **Reason for disagreement:** Referring to your user stories: ![Screenshot 2023-04-19 at 9.07.34 AM.png](https://raw.githubusercontent.com/T-Wan-Lin/pe/main/files/8e31dd48-3ce4-4f8c-8446-b73510dc5e54.png) As for what qualifies to be NotInScope: ![Screenshot 2023-04-19 at 9.12.44 AM.png](https://raw.githubusercontent.com/T-Wan-Lin/pe/main/files/02370cd5-2054-4ff1-b186-799609b4e88e.png) It is strange that you label this as NotInScope because for this feature flaw, all you have to do to improve this UI issue is to print out a `Venue Size` string before the venue size information. This does not require much additional effort and I believe that fixing it would be essential because it took several iterations of the same command to realise that the 100 was the venue size.
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]