search

Eugenetanwl3881 / pe

0 stars 0 forks source link

Some DG diagrams were hard to read as image has been stretched/compressed #16

Open Eugenetanwl3881 opened 1 year ago

Eugenetanwl3881 commented 1 year ago

For example:

image.png

nus-se-script commented 1 year ago

Team's Response

Dear Tester,

Thank you for pointing this issue out. We do not think such slightly streched words will hinder the reader, since they are still quite identifiable. The words like "DeleteBuyerCommandParser" and "DeletePetCommandParser" are clear enough. At first glance, most people will get what is written.

Thank you!

The 'Original' Bug

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

Diagram is compressed making it difficult to read the words

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

This is a cosmetic bug as it hinders the reader's ability to read the text within the diagram due to the compressed words.

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

Their Response to the 'Original' Bug

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

Dear Tester,

Thank you for pointing this issue out. We do not think such slightly streched words will hinder the reader, since they are still quite identifiable. The words like "DeleteBuyerCommandParser" and "DeletePetCommandParser" are clear enough. At first glance, most people will get what is written.

Thank you!

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: Dear Dev Team,

From a reader's point of view, the words are really compressed and hard to read (especially when viewing the pdf document at 100% zoom) Took me some time to decipher the longer words like DeleteDelivererCommandParser. Hence, I would have to disagree.