search

wz27 / pe

0 stars 0 forks source link

view command does not accept case-insensitive parameters #9

Open wz27 opened 3 years ago

wz27 commented 3 years ago

It is natural for the user to type view Timetable to switch to viewing the Timetable section which is labelled as 'Timetable' in the app. However the command view Timetable is rejected and only view timetable is allowed.

image.png

soc-se-bot commented 3 years ago

Team's Response

We feel that it is easier to type when it is all lower case, and believe that users would prefer to type without having to worry about the caps. Additionally, all our commands and flags are designed to be small letters, such as sort name desc, class 1, and we decided to standardise it throughout the application so that the user would not get confused.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This design decision was not mentioned in the User Guide. Going along with the team's reasoning, the tab labels ("Students", "Classes", "Timetable") should then be in lowercase so as not to confuse the user.