search

minzzelo / pe

0 stars 0 forks source link

Diagram font size #18

Open minzzelo opened 4 years ago

minzzelo commented 4 years ago

image.png

image.png

I believe an appropriate diagram should have the font size matching the font size of the markdown. This is too small for the users to look at

nus-pe-bot commented 3 years ago

Team's Response

We decided to stick to this diagram size as we felt that it was still readable. Making it any larger would have resulted in us having to split the diagram up which we felt would not be ideal for the readers.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This is extracted from the module website. I still stand by my point that the font size is really too small relative to the main text and readable is a subjective term. I believe that you should still match the text size in the diagram to the main text, and splitting should be done if possible. I don’t think it is ideal to the readers by just leaving the diagram like that.

image.png

image.png