search

juliussneezer04 / pe

0 stars 0 forks source link

Sequence Diagram on DG Page 19 is too small to read normally #24

Open juliussneezer04 opened 2 years ago

juliussneezer04 commented 2 years ago

Page 19 of DG could have a larger diagram

image.png

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Sequence diagrams too small

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.

image.png

image.png

image.png

image.png

This is the view at 100% and it is difficult to read. One possibility is to include less details, e.g. the CommandResult part, and also specify the font size in the puml file itself.

[original: nus-cs2103-AY2122S1/pe-interim#4770] [original labels: severity.VeryLow type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We think the font size is big enough when viewed from pdf. It is still readable. We have already extracted out most of the non-important lower level details.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: * The text size is too small for the long words that are in the diagram, e.g. updateFilteredStudentList(predicate) or FilterStudentCommand(predicate)