jonpwk / pe

0 stars 0 forks source link

Sequence Diagram in DG Too Small #15

Open jonpwk opened 2 months ago

jonpwk commented 2 months ago

The sequence diagram for execute ("/delete 240001") is too small to be viewed and requires zooming significantly.

image.png

soc-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Diagram in DG Notes is 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 The diagram is too small in the PDF version of the DG, which could hinder a reader's understanding.


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

Their Response to the 'Original' Bug

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

Sorry, our current priority is to make all diagrams correct, and we believe users can zoom in to see the image more clearly.

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.NotInScope`] - [x] I disagree **Reason for disagreement:** From the Practical Exam guide, the issue of the diagram being too small can be attributed as a VeryLow cosmetic bug. ![Screenshot 2024-04-23 at 6.43.25 PM.png](https://raw.githubusercontent.com/jonpwk/pe/main/files/cb84cc00-6b56-4214-bb9f-559a3cf6ec6d.png)