Oliverloo0909 / pe

0 stars 0 forks source link

Autocompletion command message is not formatted correctly #11

Open Oliverloo0909 opened 1 year ago

Oliverloo0909 commented 1 year ago

Screenshot 2023-04-14 150615.png

when doing /c for the autocomplete feature

  1. as seen above the possible words are not aligned
  2. 'unknow' is display, which might be puzzling to readers - dont really know what that means.
nus-se-bot commented 1 year ago

Team's Response

Unable to reproduce the centered spacing for Groceries.

Items for the Tester to Verify

:question: Issue response

Team chose [response.CannotReproduce]

Reason for disagreement: I believe the unknow "display" is clearly some sort of typo UI issue, that sticks out rather evidently. If this could not be reproduced or fixed, it should have been removed from the UI.

Not sure why it cannot be reproduced on the team's. However, i believe the app needs to be tested on all types of devices, and it happens that my one unfortunately fails.

Screenshot 2023-04-18 at 2.42.35 PM.png


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** The team does not seem to have provided a reason for the downgrade. Here is my reason for disagreeing with the downgrade: More than a cosmetic issue, I believe the typo "Unknow" is warranted to be more of a flaw, that reflects that something is clearly wrong in the UI since it's a non-existent word that makes no sense. Furthermore, as long as users enter, /c the autocompletion command, they would be met with such an issue. Hence, I believe a moderate proportion of users would face this. ![Screenshot 2023-04-18 at 2.42.35 PM.png](https://raw.githubusercontent.com/Oliverloo0909/pe/main/files/daa6867b-77f6-469b-b145-cc41cb3143de.png) Hence a low severity feels conservative and reasonable already. ![Screenshot 2023-04-18 at 3.31.53 PM.png](https://raw.githubusercontent.com/Oliverloo0909/pe/main/files/55bca151-bea1-4e26-897a-6adc35f25ed9.png)