From 7a11a71abae411439c70bda59b4d43bb40ba302f Mon Sep 17 00:00:00 2001
From: Simon Curtis <43214378+simon-curtis@users.noreply.github.com>
Date: Thu, 4 Apr 2024 17:44:30 +0000
Subject: [PATCH] Rewrite of layout and added code block styling in large view

---
 static/css/layout.css       |   1 +
 static/css/pages/lesson.css | 106 ++++++++++++++++++++++--------------
 2 files changed, 66 insertions(+), 41 deletions(-)

diff --git a/static/css/layout.css b/static/css/layout.css
index 609eeef..3de815f 100644
--- a/static/css/layout.css
+++ b/static/css/layout.css
@@ -7,4 +7,5 @@
   --gap-quarter: calc(0.25 * var(--gap));
 
   --navbar-height: calc(var(--gap-double) + 20px);
+  --border-radius: .25rem;
 }
diff --git a/static/css/pages/lesson.css b/static/css/pages/lesson.css
index 4468d56..c0d0fe7 100644
--- a/static/css/pages/lesson.css
+++ b/static/css/pages/lesson.css
@@ -1,73 +1,97 @@
+#playground {
+  display: flex;
+  flex-direction: column;
+  flex-grow: 1;
+}
+
 #left,
-#output,
-#editor {
-  min-height: 200px;
-  overflow-y: auto;
-  border-bottom: 2px solid var(--color-divider);
-  margin: 0;
-  overflow-wrap: break-word;
+#output>*,
+#editor .codeflask__flatten {
+  padding: var(--gap);
 }
 
-#left> :first-child {
-  margin-top: 0;
+#right {
+  display: flex;
+  flex-direction: column;
+  border: var(--color-divider);
+  background: var(--code-background);
+  padding-top: .15rem;
+  overflow: hidden;
+  flex-grow: 1;
+  min-height: fit-content;
+}
+
+#output,
+#editor {
+  border-top: 1px solid var(--color-accent-muted);
 }
 
 #editor {
   position: relative;
+  overflow: clip;
+  min-height: 14rem;
+  flex-grow: 1;
 }
 
-@media (min-width: 600px) {
-  #playground {
-    position: fixed;
-    top: var(--navbar-height);
-    bottom: 0;
-    left: 0;
-    right: 0;
-    padding-top: 0;
-  }
+#output {
+  min-height: 1rem;
+  background: var(--color-background-dim);
+}
 
-  #left,
-  #right {
-    width: 50vw;
-    position: absolute;
-    top: 0;
-    bottom: 0;
+#output>* {
+  margin: 0;
+  white-space: pre-wrap;
+}
+
+/* Larger then mobile */
+@media (min-width: 768px) {
+  #playground {
+    min-height: calc(100vh - var(--navbar-height));
+    flex-direction: row;
   }
 
   #left {
-    border-right: 2px solid var(--color-divider);
+    height: 100%;
+    width: 50%;
   }
 
   #right {
-    right: 0;
+    border-left: 1px solid var(--color-accent-muted);
+    width: 50%;
   }
 
-  #right> :first-child {
-    height: 62%;
+  #editor {
+    border: none;
   }
 
-  #right> :last-child {
-    height: 38%;
+  #output {
+    height: 33%;
+    overflow: auto;
+    border-top: 1px solid var(--color-accent-muted);
   }
 }
 
-#left,
-#output>*,
-#editor .codeflask__flatten {
-  padding: var(--gap);
-}
+/* Larger than medium screen and has enough to height to not worry about losing vertical space */
+@media (min-width: 1200px) and (min-height: 700px) {
+  #left {
+    /* Lift the navigation bar up a bit so it's not sitting right on the bottom*/
+    padding-bottom: calc(var(--gap) * 2);
+  }
 
-#output>* {
-  margin: 0;
-  white-space: pre-wrap;
+  #right {
+    border-left: unset;
+    border-radius: var(--border-radius);
+    padding: 2px 1px;
+    box-shadow: var(--drop-shadow);
+    /* Use calc here to add additional padding dynamically to allow for the drop shadow */
+    margin: calc(var(--gap) * 2);
+  }
 }
 
 .error,
 .warning {
-  border-width: var(--gap);
   border-style: solid;
-  border-top: 0;
-  border-bottom: 0;
+  height: 100%;
 }
 
 .error {