From 89cd1348fe349c633259ae6ce748a990fd7af0a2 Mon Sep 17 00:00:00 2001
From: rxshellg <rashell-gb@hotmail.com>
Date: Thu, 12 Feb 2026 00:46:32 -0500
Subject: [PATCH] Moved all unnecessary inline styling to CSS files

---
 .../PeopleManager/AddPersonModal.module.css   | 17 +++++++++++++
 .../PeopleManager/AddPersonModal.tsx          |  8 +++----
 src/pages/Home.module.css                     | 24 +++++++++++++++++++
 src/pages/Home.tsx                            | 22 +++++------------
 src/pages/Review.module.css                   | 22 +++++++++++++++++
 src/pages/Review.tsx                          | 19 +++++----------
 6 files changed, 78 insertions(+), 34 deletions(-)

diff --git a/src/components/PeopleManager/AddPersonModal.module.css b/src/components/PeopleManager/AddPersonModal.module.css
index a962723..fa2ec03 100644
--- a/src/components/PeopleManager/AddPersonModal.module.css
+++ b/src/components/PeopleManager/AddPersonModal.module.css
@@ -2,3 +2,20 @@
 .modalTitle {
   font-weight: bold;
 }
+
+.nameInput {
+  width: 100%;
+  margin-bottom: 16px;
+}
+
+.colorOptions {
+  display: flex;
+  gap: 8px;
+  margin-top: 8px;
+}
+
+.colorBubble {
+  width: 28px;
+  height: 28px;
+  border-radius: 50%;
+}
diff --git a/src/components/PeopleManager/AddPersonModal.tsx b/src/components/PeopleManager/AddPersonModal.tsx
index 925c009..851d0a4 100644
--- a/src/components/PeopleManager/AddPersonModal.tsx
+++ b/src/components/PeopleManager/AddPersonModal.tsx
@@ -33,19 +33,17 @@ export default function AddPersonModal({ show, onClose, onSubmit }: Props) {
         <input
           value={name}
           onChange={(e) => setName(e.target.value)}
-          style={{ width: "100%", marginBottom: 16 }}
+          className={styles.nameInput}
         />
 
         <label>Pick a color</label>
-        <div style={{ display: "flex", gap: 8, marginTop: 8 }}>
+        <div className={styles.colorOptions}>
           {DEFAULT_COLORS.map((c) => (
             <button
               key={c}
               onClick={() => setColor(c)}
+              className={styles.colorBubble}
               style={{
-                width: 28,
-                height: 28,
-                borderRadius: "50%",
                 border: color === c ? "3px solid #111" : "1px solid #aaa",
                 backgroundColor: c,
               }}
diff --git a/src/pages/Home.module.css b/src/pages/Home.module.css
index 94ae2c4..9be4e54 100644
--- a/src/pages/Home.module.css
+++ b/src/pages/Home.module.css
@@ -54,3 +54,27 @@
   flex-direction: row;
   flex-wrap: wrap;
 }
+
+.modalTitle {
+  line-height: 0.3;
+}
+
+.modalDescription {
+  margin-bottom: 12px;
+}
+
+.scanningText {
+  margin-top: 12px;
+}
+
+.errorText {
+  color: red;
+  margin-top: 8px;
+}
+
+.modalFooter {
+  display: flex;
+  gap: 12px;
+  flex-wrap: wrap;
+  align-items: center;
+}
diff --git a/src/pages/Home.tsx b/src/pages/Home.tsx
index 5a9d213..660947f 100644
--- a/src/pages/Home.tsx
+++ b/src/pages/Home.tsx
@@ -124,32 +124,22 @@ const Home = () => {
       >
         <Modal.Header closeButton>
           <Modal.Title>
-            <h2 style={{ lineHeight: 0.3 }}>Upload a receipt</h2>
+            <h2 className={styles.modalTitle}>Upload a receipt</h2>
           </Modal.Title>
         </Modal.Header>
         <Modal.Body>
-          <p style={{ marginBottom: 12 }}>
+          <p className={styles.modalDescription}>
             Try BillBuddy with your own photo, or use our sample image to see
             the flow.
           </p>
           {loading && (
-            <p style={{ marginTop: 12 }}>🔍 Scanning image for text…</p>
+            <p className={styles.scanningText}>🔍 Scanning image for text…</p>
           )}
-          {error && <p style={{ color: "red", marginTop: 8 }}>{error}</p>}
+          {error && <p className={styles.errorText}>{error}</p>}
         </Modal.Body>
         <Modal.Footer>
-          <div
-            style={{
-              display: "flex",
-              gap: 12,
-              flexWrap: "wrap",
-              alignItems: "center",
-            }}
-          >
-            <label
-              className="pinkButton"
-              style={{ margin: 0, cursor: "pointer" }}
-            >
+          <div className={styles.modalFooter}>
+            <label className="pinkButton">
               <input
                 ref={fileInputRef}
                 type="file"
diff --git a/src/pages/Review.module.css b/src/pages/Review.module.css
index e69de29..c830b6a 100644
--- a/src/pages/Review.module.css
+++ b/src/pages/Review.module.css
@@ -0,0 +1,22 @@
+.heading {
+  padding: 2rem;
+}
+
+.item {
+  margin-bottom: 1rem;
+  display: flex;
+  gap: 1rem;
+  align-items: center;
+}
+
+.quantity {
+  width: 60px;
+}
+
+.name {
+  flex: 1;
+}
+
+.price {
+  width: 100px;
+}
diff --git a/src/pages/Review.tsx b/src/pages/Review.tsx
index 126f0c0..fc70eeb 100644
--- a/src/pages/Review.tsx
+++ b/src/pages/Review.tsx
@@ -2,6 +2,7 @@ import { useLocation, useNavigate } from "react-router-dom";
 import { useState, useEffect } from "react";
 import { parseItems } from "../utils/parseReceipt";
 import type { Item } from "../types";
+import styles from "./Review.module.css";
 
 const Review = () => {
   const ocrText = useLocation().state?.ocrText || "";
@@ -59,7 +60,7 @@ const Review = () => {
   };
 
   return (
-    <div style={{ padding: "2rem" }}>
+    <div className={styles.heading}>
       <h2>Review and Edit Items</h2>
 
       {/* Empty state */}
@@ -67,22 +68,14 @@ const Review = () => {
 
       {/* Items list */}
       {items.map((item, index) => (
-        <div
-          key={item.id}
-          style={{
-            marginBottom: "1rem",
-            display: "flex",
-            gap: "1rem",
-            alignItems: "center",
-          }}
-        >
+        <div key={item.id} className={styles.item}>
           {/* Quantity */}
           <input
             type="number"
             min="1"
             value={item.quantity}
             onChange={(e) => updateItemField(index, "quantity", e.target.value)}
-            style={{ width: "60px" }}
+            className={styles.quantity}
             aria-label={`Quantity for ${item.name}`}
           />
 
@@ -91,7 +84,7 @@ const Review = () => {
             type="text"
             value={item.name}
             onChange={(e) => updateItemField(index, "name", e.target.value)}
-            style={{ flex: 1 }}
+            className={styles.name}
             aria-label="Item name"
           />
 
@@ -100,7 +93,7 @@ const Review = () => {
             type="text"
             value={`$${item.price.toFixed(2)}`}
             onChange={(e) => updateItemField(index, "price", e.target.value)}
-            style={{ width: "100px" }}
+            className={styles.price}
             aria-label={`Price for ${item.name}`}
           />