search

eliztan / pe

0 stars 0 forks source link

Potentially unnecessary capitalisation of keywords in legend #5

Open eliztan opened 2 months ago

eliztan commented 2 months ago

image.png

As input formats should be optimised to get things done fast, the unnecessarily forced capitalisation of the keywords in the Legend when inputting them might potentially annoy the user, and inconvenience them to input things quickly as they have to constantly switch capitalisations from keywords to Legend. This could potentially be solved with an .ignoreCase()

soc-se-bot commented 2 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

This app, as specified in the CS2113 instructions for TP, was designed for users who are fast at typing. While the issue you raised up sounds valid, we believe and assume that for fast typers, capitalising a letter would cause negligible hassle for them. Furthermore, we wanted to focus more on the aesthetic aspect of things in this case, and we felt that capital letters to represent items in the legend would be more appropriate and intuitive.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]