se-edu/addressbook-level3

Upgrade UML diagrams to the new version of PlantUML

Closed this issue · 6 comments

Seems the new version has some breaking changes, as the MarkBind team found (some info here) e.g., no need for GraphViz on Windows

Both diagrams and Dev Guide might needs updating.

@Eclipse-Dominator can have a look?

Do we adapt to the newer lighter-colored theme or should we preserve the old color scheme?

The most noticeable theme change is the border color change as well as the background change

image


For now I will use the old theme version.

@Eclipse-Dominator I think there is no good reason to keep the old style. We can go with the new default style.

I think it might be better to stick with the old theme / rose theme for now as some of the color scheme changes in style.puml as some of the diagrams are based on the old color scheme.

I think it might be better to stick with the old theme / rose theme for now as some of the color scheme changes in style.puml as some of the diagrams are based on the old color scheme.

I see. I would like to see the difference before making the decision. It is ideal if new batches of student's don't need to follow the legacy stuff at all, as they are unnecessary complications for them.

I will provide a separate PR to display the changes in color schemes