search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incorrect arrowheads used in all SDs #99

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

From course website expected sequence diagrams use these arrow heads

image.png

image.png

image.png

image.png

image.png

image.png

image.png

image.png

To be specific this is what I refer to as an arrowhead

image.png

[original: nus-cs2113-AY2425S1/pe-interim#195] [original labels: severity.Low type.DocumentationBug]

RCPilot1604 commented 1 week ago

Team's Response

Hi, thank you for your bug report!

We agree that the arrow heads should be the one that follows the course website. However, the choice of this arrow head does not confuse the developer reading the diagrams as the arrow head appears to be similar to a filled arrow head.

As this is a cosmetic issue, the issue will be labelled as VeryLow. Thank you!

Duplicate status (if any):

--