dcchan98 / pe

0 stars 0 forks source link

Image a bit too small. May want to reference UML diagram or factorise out some details #12

Open dcchan98 opened 3 years ago

dcchan98 commented 3 years ago

No details provided. Screenshot 2020-11-13 at 5.24.05 PM.png

For example, parser can be taken out because every command implements this parser. Hence a single diagram explaining parser would suffice.

nus-se-bot commented 3 years ago

Team's Response

Parser is drawn so the diagram is consistent with other diagrams. The font size is around the other diagrams' font size, but a bit smaller. I change this to very low since the words can still be seen, and the user can always zoom in the picture or open it in another tab with a bigger size.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]