search

LJ-37 / pe

0 stars 0 forks source link

Cramped and messy class diagram #12

Open LJ-37 opened 3 years ago

LJ-37 commented 3 years ago

Class diagram is too cramped, making the arrows in the diagram to cross one another and hard to follow. Would suggest splitting the diagram into multiple diagrams to show the class section by section. image.png

nus-pe-bot commented 3 years ago

Team's Response

UML diagram seems to serve its purpose to a good degree and certain aspects are already split up. Accepted because it might cause some users minor problems in viewing although problem seems to be only cosmetic.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: Severity should be low as it affects the user the usage ie. affects the user understanding when reading. As stated in the response, it might cause minor problems which fits the definition of a low bug instead of a very low bug.

image.png

image.png