search

azriellee / pe

0 stars 0 forks source link

View feature sequence diagram #26

Open azriellee opened 1 year ago

azriellee commented 1 year ago

image.png Sequence diagram too small and thus not comprehensible

nus-pe-bot commented 1 year ago

Team's Response

Separating diagram into different parts would be more confusing so we decided to compromise.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason] This severely hinders readability of the diagram, it does not need to be separated into different parts, just enlarged. The following is how the diagram looks on a normal 100% zoom ratio.

image.png

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** [replace this with your reason] This is not an aesthetics issue, this hinders readability of the diagram thus will reduce the understanding of the system the developer will have.