nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Format: Separation Display #450

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

Screenshot 2024-11-15 at 16.24.18.png If the list is empty, there is no display for separation line in the command interface, which leads to a mess when display the next command.


[original: nus-cs2113-AY2425S1/pe-interim#716] [original labels: type.FeatureFlaw severity.Low]

KSanjith commented 1 week ago

Team's Response

Dear Tester, we would like to reject your issue. We would also lower it's severity to VeryLow.

We intentionally omitted the dashed lines for all 1 liner error messages related to empty lists. Given how short the error messages are in this case, adding 2 more lines to a 1 liner error message would just add more clutter to the UI in our opinion. Hence we would reject your issue as this is an intended intentional feature on our part.

At worst, this is a very minor graphical issue that makes next to no difference to the user in a CLI environment. Given that it is also purely cosmetic, it should also be VeryLow severity. Though we still reject this issue in it's entirety.

Duplicate status (if any):

--