search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

massblist b/ should correct for case typos #1612

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Currently, massblist prefix b/ does not correct for case typos. This is bad for fast typists who might not remember the case of the input required.

image.png

[original: nus-cs2103-AY2021S2/pe-interim#1612] [original labels: severity.Low type.FeatureFlaw]

IceBear789 commented 3 years ago

Team's Response

Reason for rejection: Both the mass blacklist and mode of contact commands are designed to be case sensitive in order to be consistent with the rest of the software. (Observe that no command in our software correct for case sensitivity) Thus "not remembering the case of the input required" is not an issue here as none of the command keywords require uppercase. Furthermore, if your suggestion is to be implemented, every other command in the software will also have to be changed to maintain consistency. (This is a non-trivial change that requires extensive regression testing)

Another point to note here is that for a fast typist, typing in lower case is still the fastest way to type. Hence this suggestion will not have a significant improvement on the user's speed anyway.

Duplicate status (if any):

--