nus-pe-bot
commented
3 years ago Team's Response
General response: By making the diagram bigger to make the font size bigger, the image will be too huge.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: Thank you for your response, however I disagree.
Firstly, I think the zoomed in screenshot does not do the severity of the issue justice. Here's a version of the same screenshot that is not zoomed in (at 100% zoom).
As you can see, the words in the diagram are barely visible. As reader, I had to zoom in to see the text, which was very inconvenient. To me, it is important that the user can read the diagrams without having to zoom in. If one is unable to make the diagram fit, there are many ways to rectify this, such as omitting unnecessary details or splitting the diagram into parts.
Secondly, hard to read is counted as a DG bug. We also went through in the tutorial on DG bugs that text should match the text size of the diagrams (ex2) and diagrams should not be too small because they will be unreadable (ex12).
Lastly, while the team claims that By making the diagram bigger to make the font size bigger, the image will be too huge. I cloned the repo and spent a few minutes playing around with the diagram and managed to get this result.
In this image (also at 100% zoom), the labels are much more readable, and I did not have to omit any details. Hence, I disagree that it is not possible to make the font size bigger.
Because of the above reasons, I feel this should be counted as a bug.
The font size of the sequence diagram is too small and not consistent with other text in the document making it very difficult to read.