search

andrewbaruch / pe

0 stars 0 forks source link

Small diagram text #7

Open andrewbaruch opened 2 years ago

andrewbaruch commented 2 years ago

The text in this diagram is pretty small compared to the rest of the file

Screenshot (34).png

nus-pe-bot commented 2 years ago

Team's Response

Our group has already abstracted out the low-level details regarding the creation of an AddContactCommand. Given that the sequence diagram aims to show how an addcontact operation works, we are already giving the bare minimum when it comes to information regarding the execution of an addcontact operation. More importantly, perhaps the tester had two windows open side by side while reviewing our DG but when in fullscreen (regular A4 size), the text is certainly readable, as is evident in the screenshot.

Screenshot 2021-11-14 at 12.25.05 AM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]