Refactor Frame usage and improve HTML semantics

* Extract inline guide frames into a reusable Frame component
* Update step elements to use proper ordered lists (<ol> and <li>)
* Move related image and list styling to Frame.css

src/App.tsx

@@ -2,35 +2,15 @@ import "./styles/reset.css";
 import "./styles/App.css";
 import { Outlet } from "react-router";
 import { Link } from "react-router";
-import { useGuides } from "./lib/contexts/guide.context";
-import { NavLink } from "react-router";
+import Navbar from "./components/navbar/Navbar";
 
 function App() {
-  const { guides } = useGuides();
 
   return (
     <>
       <header>
-        <h1 className="main-title">
-          <Link to="/">SnapSteps</Link>
-        </h1>
-        <nav>
-          <Link className="create-new-guide-btn" to="/new-guide">
-            Create New Guide
-          </Link>
-
-          <h2>Guides</h2>
-          <ul className="guide-list">
-            {guides.map((guide) => (
-              <li key={guide.id}>
-                <NavLink to={`/guide/${guide.id}`}>{guide.title}</NavLink>
-              </li>
-            ))}
-          </ul>
-          <NavLink className="settings-btn" to="/settings">
-            Settings
-          </NavLink>
-        </nav>
+        <h1><Link to="/">SnapSteps</Link></h1>
+        <Navbar />
       </header>
 
       <main>

src/components/frame/Frame.tsx

@@ -0,0 +1,18 @@
+import type { FrameType } from "../../lib/schemas";
+
+export default function Frame({ frame } : { frame: FrameType }) {
+  return (
+    <div className="frame">
+      <h2>{frame.title}</h2>
+      <img src={URL.createObjectURL(frame.file)} alt={frame.title} />
+
+      <ol className="steps">
+        {frame.steps.map((step, stepIndex) => (
+          <li key={stepIndex} className="step">
+            {step.text}
+          </li>
+        ))}
+      </ol>
+    </div>
+  );
+}

src/components/navbar/Navbar.css

@@ -0,0 +1,50 @@
+nav {
+    display: flex;
+    flex-direction: column;
+    height: 100%;
+    padding-bottom: var(--size-lg);
+    width: 100%;
+
+    a {
+        padding: 0.6rem var(--size-md) 0.5rem;
+        border-radius: var(--radius);
+        line-height: 1;
+        display: block;
+    }
+
+    .settings-button {
+        margin-top: auto;
+        align-self: center;
+        font-weight: bold;
+    }
+
+    .settings-button.active {
+        margin-top: auto;
+        align-self: center;
+        color: var(--color-accent);
+    }
+
+    h2 {
+        margin-top: var(--size-lg);
+        font-size: 1rem;
+        opacity: 0.5;
+    }
+    li {
+        width: 100%;
+    }
+    .guide-list a {
+        transition:
+            background-color 0.125s,
+            border-left 0.125s;
+        background-color: #fff;
+        border-left: 4px solid var(--color-border);
+    }
+    .guide-list a.active {
+        background-color: #fff;
+        border-left: 4px solid var(--color-accent);
+    }
+    .guide-list a:hover:not(.active) {
+        border-left: 4px solid var(--color-accent-hover);
+        opacity: 0.8;
+    }
+}

src/components/navbar/Navbar.tsx

@@ -0,0 +1,19 @@
+import { Link, NavLink } from "react-router";
+import "./Navbar.css"
+import GuideList from "./guide-list/GuideList";
+
+export default function Navbar() {
+  return (
+    <nav>
+      <Link className="button primary-button" to="/new-guide">
+        Create New Guide
+      </Link>
+
+      <h2>Guides</h2>
+      <GuideList />
+      <NavLink className="button secondary-button settings-button" to="/settings">
+        Settings
+      </NavLink>
+    </nav>
+    );
+}

src/components/navbar/guide-list/GuideList.css

@@ -0,0 +1,8 @@
+    .guide-list {
+        list-style: none;
+        padding: 0;
+        margin-top: var(--size-sm);
+        display: flex;
+        flex-direction: column;
+        gap: var(--size-sm);
+    }

src/components/navbar/guide-list/GuideList.tsx

@@ -0,0 +1,19 @@
+import { NavLink } from "react-router";
+import "./GuideList.css";
+import { useGuides } from "../../../lib/contexts/guide.context";
+import type { GuideType } from "../../../lib/schemas";
+
+export default function GuideList() {
+  const { guides } = useGuides();
+
+
+  return (
+    <ul className="guide-list">
+      {guides.map((guide: GuideType) => (
+        <li key={guide.id}>
+          <NavLink to={`/guide/${guide.id}`}>{guide.title}</NavLink>
+        </li>
+      ))}
+    </ul>
+  );
+}

src/features/guides/components/Frame.css

@@ -8,6 +8,12 @@
     border-radius: var(--radius);
 }
 
+.frame img {
+    max-height: 350px;
+    object-fit: contain;
+    object-position: left center;
+}
+
 .frame .buttons {
     display: flex;
     gap: var(--size-md);
@@ -20,3 +26,7 @@
 .frame .buttons .remove {
     margin-left: auto;
 }
+
+.frame ol {
+    margin-left: var(--size-md);
+}

src/features/guides/components/Frame.tsx

@@ -72,7 +72,7 @@ export default function Frame({
       >
         Add Step
       </button>
-      <div className="steps">
+      <ol className="steps">
         {frame.steps.map((step: StepType, index: number) => (
           <Step
             key={step.id}
@@ -82,7 +82,7 @@ export default function Frame({
             updateStep={updateStep}
           />
         ))}
-      </div>
+      </ol>
     </div>
   );
 }

src/features/guides/components/Step.tsx

@@ -13,7 +13,7 @@ export default function Step({
   index: number;
 }) {
   return (
-    <div className={"step " + index}>
+    <li className={"step " + index}>
       <label>
         <span>{index + 1}. </span>
         <input
@@ -25,6 +25,6 @@ export default function Step({
       <button type="button" onClick={() => removeStep(step.id)}>
         Remove Step
       </button>
-    </div>
+    </li>
   );
 }

src/features/guides/routes/Guide.css

@@ -3,11 +3,13 @@ main:has(.guide) {
     display: flex;
     justify-content: center;
 }
+
 .guide__frames {
     display: flex;
     flex-direction: column;
     gap: var(--size-xl);
 }
+
 .guide {
     max-width: 600px;
     .delete-button {
@@ -15,40 +17,30 @@ main:has(.guide) {
         bottom: var(--size-md);
         right: var(--size-md);
     }
+
     .edit-button {
         position: absolute;
         top: var(--size-md);
         right: 8rem;
     }
+
     .export-button {
         position: absolute;
         top: var(--size-md);
         right: var(--size-md);
     }
-    .guide__frame {
-        padding: var(--size-md);
-        background-color: var(--color-bg-panel);
-        display: flex;
-        flex-direction: column;
-        gap: var(--size-sm);
-    }
-    .guide__frame img {
-        max-height: 350px;
-        object-fit: contain;
-        object-position: left center;
-    }
-    .guide__frame ol {
-        margin-left: var(--size-md);
-    }
+
 }
 @media print {
     main:has(.guide) {
         padding: 0rem;
         justify-content: flex-start;
     }
+
     .guide__frames {
         display: block;
     }
+
     .guide__frame {
         position: relative;
         break-inside: avoid;
@@ -56,6 +48,7 @@ main:has(.guide) {
         background-color: var(--color-bg-panel);
         margin: var(--size-lg) 0;
     }
+
     .guide__frame img {
         max-width: 100%;
         max-height: 100px;

src/features/guides/routes/Guide.tsx

@@ -2,6 +2,7 @@ import "./Guide.css";
 import { useParams, Link } from "react-router";
 import { useGuides } from "../../../lib/contexts/guide.context";
 import useGuide from "../hooks/useGuide";
+import Frame from "../../../components/frame/Frame";
 
 export default function Guide() {
   const { id } = useParams();
@@ -20,18 +21,7 @@ export default function Guide() {
 
       <div className="guide__frames">
         {guide.frames.map((frame, index) => (
-          <div key={index} className="guide__frame frame">
-            <h2>{frame.title}</h2>
-            <img src={URL.createObjectURL(frame.file)} alt={frame.title} />
-
-            <ol className="steps">
-              {frame.steps.map((step, stepIndex) => (
-                <li key={stepIndex} className="guide__step">
-                  {step.text}
-                </li>
-              ))}
-            </ol>
-          </div>
+          <Frame key={index} frame={frame} />
         ))}
       </div>
 

src/styles/App.css

@@ -1,12 +1,13 @@
 :root {
     /* color */
-    --color-border: #ccc;
-    --color-bg-sidebar: #eee;
-    --color-bg-panel: #fafafa;
-    --color-accent: #ff7700;
-    --color-accent-hover: #ff9900;
-    --color-accent-border: #ff770080;
-    --color-accent-bg-subtle: #ff770010;
+    --color-border: oklch(87.1% 0.006 286.286);
+    --color-bg-sidebar: oklch(92% 0.004 286.32);
+    --color-bg-panel: oklch(96.7% 0.001 286.375);
+
+    --color-accent: oklch(70.5% 0.213 47.604);
+    --color-accent-hover: oklch(75% 0.183 55.934);
+    --color-accent-border: oklch(83.7% 0.128 66.29);
+    --color-accent-bg-subtle: oklch(98% 0.016 73.684);
 
     /* layout */
     --sidebar-width: 250px;
@@ -19,19 +20,20 @@
     --size-md: 1rem;
     --size-lg: 2rem;
     --size-xl: 3rem;
+    --size-2xl: 4rem;
+    --size-3xl: 5rem;
 }
+
 html,
 body {
     font-family: system-ui, Arial, sans-serif;
     position: relative;
 }
+
 #root {
     min-height: 100vh;
 }
-.app-body {
-    display: flex;
-    flex: 1;
-}
+
 header {
     position: fixed;
     height: 100%;
@@ -43,33 +45,19 @@ header {
     gap: var(--size-sm);
     border-right: 3px solid var(--color-border);
     background-color: var(--color-bg-sidebar);
-    .main-title a {
+
+    h1 a {
         text-decoration: none;
         color: inherit;
     }
-    .create-new-guide-btn {
-        padding: var(--size-sm) var(--size-md);
-        border-radius: var(--radius);
-        background-color: var(--color-accent);
-        color: #fff;
-        text-decoration: none;
-        font-weight: bold;
-        text-align: center;
-    }
-    .guide-list {
-        list-style: none;
-        padding: 0;
-        margin-top: var(--size-sm);
-        display: flex;
-        flex-direction: column;
-        gap: var(--size-sm);
-    }
 }
+
 main {
     margin-left: var(--sidebar-width);
     flex: 1;
     padding: var(--size-md);
 }
+
 footer {
     position: fixed;
     bottom: 0;
@@ -79,52 +67,9 @@ footer {
     text-align: center;
     width: var(--sidebar-width);
 }
-nav {
-    display: flex;
-    flex-direction: column;
-    height: 100%;
-    padding-bottom: var(--size-lg);
-    width: 100%;
-    a {
-        padding: 0.6rem var(--size-md) 0.5rem;
-        border-radius: var(--radius);
-        line-height: 1;
-        display: block;
-    }
-    .settings-btn {
-        margin-top: auto;
-        align-self: center;
-        font-weight: bold;
-    }
-    .settings-btn.active {
-        margin-top: auto;
-        align-self: center;
-        color: var(--color-accent);
-    }
-    h2 {
-        margin-top: var(--size-lg);
-        font-size: 1rem;
-        opacity: 0.5;
-    }
-    li {
-        width: 100%;
-    }
-    .guide-list a {
-        transition:
-            background-color 0.125s,
-            border-left 0.125s;
-        background-color: #fff;
-        border-left: 4px solid var(--color-border);
-    }
-    .guide-list a.active {
-        background-color: #fff;
-        border-left: 4px solid var(--color-accent);
-    }
-    .guide-list a:hover:not(.active) {
-        border-left: 4px solid var(--color-accent-hover);
-        opacity: 0.8;
-    }
-}
+
+
+
 ul {
     padding: 0;
     display: flex;
@@ -132,6 +77,7 @@ ul {
     gap: var(--size-xs);
     list-style: none;
 }
+
 a,
 input,
 button,
@@ -141,9 +87,31 @@ textarea {
     font-size: inherit;
     color: inherit;
 }
+
 a {
     text-decoration: none;
 }
+
+.primary-button {
+    padding: var(--size-sm) var(--size-md);
+    border-radius: var(--radius);
+    background-color: var(--color-accent);
+    color: #fff;
+    text-decoration: none;
+    font-weight: bold;
+    text-align: center;
+}
+
+.secondary-button {
+    padding: var(--size-sm) var(--size-md);
+    border-radius: var(--radius);
+    background-color: var(--color-bg-sidebar);
+    color: #000;
+    text-decoration: none;
+    font-weight: bold;
+    text-align: center;
+}
+
 input,
 textarea,
 .file-upload {