search

nus-cs2113-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Userguide scetion 5.3.3 - add timetable #184

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

I feel that output should only show intented output. Any explanation could be done under lets say directly under the chaining commands title.

image.png

[original: nus-cs2113-AY2021S1/pe-interim#167]

fchensan commented 3 years ago

Team's Response

Assuming by "explanation" the tester meant the: (The indentation of each line of the ... part, we think that it is actually correct to put it under output, since we are explaining what the output in the screenshot means. Putting it directly under the "chaining commands" title as the tester suggested makes it less clear and harder to read, since the screenshot and the explanation would be separated.

Duplicate status (if any):

--