nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Not efficient for fast CLI users #1760

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

For your createtut command, I feel that createtut might be quite long for a command especially when the app is meant for fast typers CLI based. Perhaps a shorter command could be just tut (To show that you're creating a new tutorial or if tut is not intuitive enough, perhaps try addtut instead).


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

BlazeChron commented 1 week ago

Team's Response

As a team, we set a 10-character limit for any new commands we added on top of the AB3 implementation. 10-characters is what we felt that reasonably fast-typists are fine with typing.

For createtut specifically, we felt that alternatives such as addtut and tut could be confused as commands to enroll a student in a tutorial. Therefore, we chose createtut as it is more descriptive and within our 10-character limit.

We could further optimise by reducing the length of the command in the future, but we believe it is not the most important feature that we should have now, so it is classified as notInScope

Duplicate status (if any):

--