search

securespider / pe

0 stars 0 forks source link

Improperly sized sequence diagram #26

Open securespider opened 1 year ago

securespider commented 1 year ago

Diagram is way too small, the words cannot be read easily

image.png

soc-se-bot commented 1 year ago

Team's Response

The diagram is readable when zoomed in.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

"While the reader can zoom to see smaller details, this can still be considered a cosmetic issue" The characters in the image is smaller than the words in the rest of the document.

There is too much info in the diagram as there is simply no need to have sort be called multiple times if it's finally being implemented in the UniquePersonList class.