search

JodyLorah / pe

0 stars 0 forks source link

Counter intuitive commands #5

Open JodyLorah opened 3 years ago

JodyLorah commented 3 years ago

There seems to be 2 means to blacklist someone. First is to go to the individual index and undo whatever is there (blacklist to unblacklist, vice versa), and second there is the means to mass blacklist/unblacklist people. It is quite counter intuitive to do the former since the user has to go and check whether the person is currently blacklisted before blacklisting the person. This is an extra step which can be eradicated by the already implemented second method, making the app slightly less user friendly.

image.png

nus-pe-bot commented 3 years ago

Team's Response

For the case where the second method is used to blacklist 1 person, the user still has to look for the person in the list to get the index. The user would also see the blacklist status along with the index.

Then, if the person is already blacklisted, the case ends for both methods. However, if the person is un-blacklisted, the command to input would be blist INDEX vs massblist INDEX-INDEX b/unblacklist and it is clear the former is faster and more convenient to type. (Note that this example command for massblacklist is currently invalid due to the existence of the blacklist feature)

It is true that the user has to remember more commands, however this is an acceptable tradeoff for the efficiency that having 2 commands provides.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Still disagree since the user still has to go through one more step of processing whether the current user is blacklisted before deciding whether to blacklist them or do nothing. Either way the former still causes more hassles and is more counter intuitive.