freddychenyouren2 / pe

0 stars 0 forks source link

DG - UI Class Diagram can be neater #10

Open freddychenyouren2 opened 7 months ago

freddychenyouren2 commented 7 months ago

The following is ProjectPRO's UI Class Diagram in the Developer Guide

image.png

The arrows in the lower right part of the diagram could have been neater by combining the overlapping arrows inside the .puml file for the UI Class Diagram. (Something similar to skinparam group Inheritance)

nus-se-script commented 7 months ago

Team's Response

We understand that the arrows are arranged in a messy manner in the diagram. However, this is a limitation of plantuml as images are autogenerated from software. Thus, we were unable to make the diagram tidier.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [Thank you for your response. Kindly see the image below on UI Class Diagram where multiple arrows are combined to the UiPart class. There could be arrows from the original UI Class Diagram that are 'hidden' (They have [hidden] in the arrow) which cause challenges in modifying your UI Class Diagram. But from the images below, it can be seen that combining the arrows is possible using skinparam groupInheritance, modifying the 'hidden' arrows, or potentially shifting the diagram around.

Hope this helps!]

image.png

image.png