From a164c6c77d67c4622cb393c3a92b19acf371a58e Mon Sep 17 00:00:00 2001
From: j-james
Date: Fri, 2 Dec 2022 21:52:50 -0800
Subject: Phase 4: Task 3

---
 README.md              |   5 ++++-
 UML_Design_Diagram.png | Bin 0 -> 130843 bytes
 2 files changed, 4 insertions(+), 1 deletion(-)
 create mode 100644 UML_Design_Diagram.png

diff --git a/README.md b/README.md
index 850b7a9..332e716 100644
--- a/README.md
+++ b/README.md
@@ -42,7 +42,10 @@ Removed tab /home/apropos/Projects/website/j-james/index.html from tablist
 
 ## Phase IV: Task 3
 
-
+I think that the design structure shown by the UML diagram is overall pretty much fine, especially for a student project.
+There are definitely some problems with it / things I'd do differently, however:
+- The BrowserBar and BrowserCanvas have a mutual dependency with BrowserWindow: this doesn't make too much sense and was done because I needed to access BrowserWindow methods from within them. If I did it again, I would either nest the classes or use the Singleton design principle.
+- The CssParser is not connected to anything else, as it went unused. I would like to use that class in the future - but I never fully finished implementing the HTML layout, so it didn't make sense to apply CSS. (The JsonUtils class is also unconnected, but that's just because it consists of only static methods.)
 
 ## credits
 
diff --git a/UML_Design_Diagram.png b/UML_Design_Diagram.png
new file mode 100644
index 0000000..0a46b96
Binary files /dev/null and b/UML_Design_Diagram.png differ
-- 
cgit v1.2.3-70-g09d2