mandykqh / pe

0 stars 0 forks source link

Diagram text too small #19

Open mandykqh opened 1 year ago

mandykqh commented 1 year ago

Compared with the content text

Screenshot 2023-04-14 at 3.32.04 PM.png

soc-pe-bot commented 1 year ago

Team's Response

Does not affect quality of DG, and it is still relatively visible.

The 'Original' Bug

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

Edit Command (Sequence Diagram) visual is too small in DG

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

The font used in the sequence diagram is too small as compared to the font used throughout the DG. This is hard to read for the reader.


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

Their Response to the 'Original' Bug

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

Accepted as cosmetics because the text is too small.

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`] - [x] I disagree **Reason for disagreement:** It is marked as a duplicate bug but it should still be marked as accepted (as claimed by the team's response in the original bug). This is considered a cosmetic flaw in this module as seen below. As quoted in the textbook: ![image.png](https://raw.githubusercontent.com/mandykqh/pe/main/files/0c37a95b-7861-4845-8fcd-a1e9192123e5.png) According to Tutorial 10: ![Screenshot 2023-04-20 at 12.44.02 PM.png](https://raw.githubusercontent.com/mandykqh/pe/main/files/3088f013-98d0-4a78-9baf-0985921db631.png)