MarcusTXK / pe

0 stars 0 forks source link

Barely legible text for diagrams #12

Open MarcusTXK opened 3 years ago

MarcusTXK commented 3 years ago

Expected:

Actual:

More examples: image.png

image.png

nus-pe-bot commented 3 years ago

Team's Response

1) This is purely cosmetic

2) WRT the sequence diagram, while in most cases we would agree, for these two the process is already explained in detail before the diagram. All the parts in the diagram are 'essential details' as it is intended to be a low-level illustration of the way the command works; splitting the diagram into 2 will obstruct the meaning even more. (As you can see, it is a single command)

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: ### Reasons for disagreeing with team's response

Rationale for severity

Example of me trying to read the sequence diagram after zooming in till the text in the sequence diagram roughly matches the original font size of the rest of the DG text: image.png