From eb087e457c3b2c1dfafa9cd5a79b90a20e74f843 Mon Sep 17 00:00:00 2001
From: ribardej <ribardej@student.cvut.cz>
Date: Wed, 15 Oct 2025 10:06:22 +0200
Subject: [PATCH] feat(frontend): improved and centered UI

---
 7project/frontend/src/App.css                 |  43 +----
 7project/frontend/src/api.ts                  |   2 +-
 7project/frontend/src/main.tsx                |   1 +
 7project/frontend/src/pages/Dashboard.tsx     | 155 +++++++++---------
 .../frontend/src/pages/LoginRegisterPage.tsx  |  78 +++++----
 5 files changed, 127 insertions(+), 152 deletions(-)

diff --git a/7project/frontend/src/App.css b/7project/frontend/src/App.css
index b9d355d..c776d1c 100644
--- a/7project/frontend/src/App.css
+++ b/7project/frontend/src/App.css
@@ -1,42 +1 @@
-#root {
-  max-width: 1280px;
-  margin: 0 auto;
-  padding: 2rem;
-  text-align: center;
-}
-
-.logo {
-  height: 6em;
-  padding: 1.5em;
-  will-change: filter;
-  transition: filter 300ms;
-}
-.logo:hover {
-  filter: drop-shadow(0 0 2em #646cffaa);
-}
-.logo.react:hover {
-  filter: drop-shadow(0 0 2em #61dafbaa);
-}
-
-@keyframes logo-spin {
-  from {
-    transform: rotate(0deg);
-  }
-  to {
-    transform: rotate(360deg);
-  }
-}
-
-@media (prefers-reduced-motion: no-preference) {
-  a:nth-of-type(2) .logo {
-    animation: logo-spin infinite 20s linear;
-  }
-}
-
-.card {
-  padding: 2em;
-}
-
-.read-the-docs {
-  color: #888;
-}
+/* App-level styles moved to ui.css for a cleaner layout. */
diff --git a/7project/frontend/src/api.ts b/7project/frontend/src/api.ts
index ff2d679..95933d4 100644
--- a/7project/frontend/src/api.ts
+++ b/7project/frontend/src/api.ts
@@ -88,7 +88,7 @@ export async function createTransaction(input: CreateTransactionInput): Promise<
 }
 
 export async function getTransactions(): Promise<Transaction[]> {
-  const res = await fetch(`${getBaseUrl()}/transactions/`, {
+    const res = await fetch(`${getBaseUrl()}/transactions/`, {
     headers: { 'Content-Type': 'application/json', ...authHeaders() },
   });
   if (!res.ok) throw new Error('Failed to load transactions');
diff --git a/7project/frontend/src/main.tsx b/7project/frontend/src/main.tsx
index bef5202..d8e0921 100644
--- a/7project/frontend/src/main.tsx
+++ b/7project/frontend/src/main.tsx
@@ -1,6 +1,7 @@
 import { StrictMode } from 'react'
 import { createRoot } from 'react-dom/client'
 import './index.css'
+import './ui.css'
 import App from './App.tsx'
 
 createRoot(document.getElementById('root')!).render(
diff --git a/7project/frontend/src/pages/Dashboard.tsx b/7project/frontend/src/pages/Dashboard.tsx
index f606fdb..2c6b05f 100644
--- a/7project/frontend/src/pages/Dashboard.tsx
+++ b/7project/frontend/src/pages/Dashboard.tsx
@@ -36,9 +36,7 @@ export default function Dashboard({ onLogout }: { onLogout: () => void }) {
     }
   }
 
-  useEffect(() => {
-    loadAll();
-  }, []);
+  useEffect(() => { loadAll(); }, []);
 
   const last10 = useMemo(() => {
     const sorted = [...transactions].sort((a, b) => b.id - a.id);
@@ -56,9 +54,7 @@ export default function Dashboard({ onLogout }: { onLogout: () => void }) {
     return arr;
   }, [last10, minAmount, maxAmount, filterCategoryId, searchText]);
 
-  function categoryNameById(id: number) {
-    return categories.find(c => c.id === id)?.name || `#${id}`;
-  }
+  function categoryNameById(id: number) { return categories.find(c => c.id === id)?.name || `#${id}`; }
 
   async function handleCreate(e: React.FormEvent) {
     e.preventDefault();
@@ -71,85 +67,90 @@ export default function Dashboard({ onLogout }: { onLogout: () => void }) {
     try {
       const created = await createTransaction(payload);
       setTransactions(prev => [created, ...prev]);
-      // reset form
-      setAmount('');
-      setDescription('');
-      setSelectedCategoryId('');
+      setAmount(''); setDescription(''); setSelectedCategoryId('');
     } catch (err: any) {
       alert(err?.message || 'Failed to create transaction');
     }
   }
 
   return (
-    <div style={{ maxWidth: 900, margin: '2rem auto', padding: 16 }}>
-      <header style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', marginBottom: 16 }}>
-        <h2 style={{ margin: 0 }}>Dashboard</h2>
-        <button onClick={onLogout}>Logout</button>
-      </header>
-
-      <section style={{ border: '1px solid #eee', padding: 12, borderRadius: 8, marginBottom: 16 }}>
-        <h3 style={{ marginTop: 0 }}>Add Transaction</h3>
-        <form onSubmit={handleCreate} style={{ display: 'flex', gap: 8, flexWrap: 'wrap' }}>
-          <input type="number" step="0.01" placeholder="Amount" value={amount} onChange={(e) => setAmount(e.target.value)} required />
-          <input type="text" placeholder="Description (optional)" value={description} onChange={(e) => setDescription(e.target.value)} />
-          <select value={selectedCategoryId} onChange={(e) => setSelectedCategoryId(e.target.value ? Number(e.target.value) : '')}>
-            <option value="">No category</option>
-            {categories.map(c => (
-              <option key={c.id} value={c.id}>{c.name}</option>
-            ))}
-          </select>
-          <button type="submit">Add</button>
-        </form>
-      </section>
-
-      <section style={{ border: '1px solid #eee', padding: 12, borderRadius: 8, marginBottom: 16 }}>
-        <h3 style={{ marginTop: 0 }}>Filters</h3>
-        <div style={{ display: 'flex', gap: 8, flexWrap: 'wrap' }}>
-          <input type="number" step="0.01" placeholder="Min amount" value={minAmount} onChange={(e) => setMinAmount(e.target.value)} />
-          <input type="number" step="0.01" placeholder="Max amount" value={maxAmount} onChange={(e) => setMaxAmount(e.target.value)} />
-          <select value={filterCategoryId} onChange={(e) => setFilterCategoryId(e.target.value ? Number(e.target.value) : '')}>
-            <option value="">All categories</option>
-            {categories.map(c => (
-              <option key={c.id} value={c.id}>{c.name}</option>
-            ))}
-          </select>
-          <input type="text" placeholder="Search in description" value={searchText} onChange={(e) => setSearchText(e.target.value)} />
+    <div className="app-layout">
+      <aside className="sidebar">
+        <div className="logo">7Project</div>
+        <nav className="nav">
+          <button className="active">Home</button>
+          <button>Account</button>
+          <button>Appearance</button>
+        </nav>
+      </aside>
+      <div className="content">
+        <div className="topbar">
+          <h2 style={{ margin: 0 }}>Dashboard</h2>
+          <div className="actions">
+            <span className="user muted">Signed in</span>
+            <button className="btn" onClick={onLogout}>Logout</button>
+          </div>
         </div>
-      </section>
+        <main className="page space-y">
+          <section className="card">
+            <h3>Add Transaction</h3>
+            <form onSubmit={handleCreate} className="form-row">
+              <input className="input" type="number" step="0.01" placeholder="Amount" value={amount} onChange={(e) => setAmount(e.target.value)} required />
+              <input className="input" type="text" placeholder="Description (optional)" value={description} onChange={(e) => setDescription(e.target.value)} />
+              <select className="input" value={selectedCategoryId} onChange={(e) => setSelectedCategoryId(e.target.value ? Number(e.target.value) : '')}>
+                <option value="">No category</option>
+                {categories.map(c => (<option key={c.id} value={c.id}>{c.name}</option>))}
+              </select>
+              <button className="btn primary" type="submit">Add</button>
+            </form>
+          </section>
 
-      <section style={{ border: '1px solid #eee', padding: 12, borderRadius: 8 }}>
-        <h3 style={{ marginTop: 0 }}>Latest Transactions (last 10)</h3>
-        {loading ? (
-          <div>Loading…</div>
-        ) : error ? (
-          <div style={{ color: 'crimson' }}>{error}</div>
-        ) : filtered.length === 0 ? (
-          <div>No transactions</div>
-        ) : (
-          <table style={{ width: '100%', borderCollapse: 'collapse' }}>
-            <thead>
-              <tr>
-                <th style={{ textAlign: 'left', borderBottom: '1px solid #ddd' }}>ID</th>
-                <th style={{ textAlign: 'right', borderBottom: '1px solid #ddd' }}>Amount</th>
-                <th style={{ textAlign: 'left', borderBottom: '1px solid #ddd' }}>Description</th>
-                <th style={{ textAlign: 'left', borderBottom: '1px solid #ddd' }}>Categories</th>
-              </tr>
-            </thead>
-            <tbody>
-              {filtered.map(t => (
-                <tr key={t.id}>
-                  <td style={{ padding: '6px 4px' }}>{t.id}</td>
-                  <td style={{ padding: '6px 4px', textAlign: 'right' }}>{formatAmount(t.amount)}</td>
-                  <td style={{ padding: '6px 4px' }}>{t.description || ''}</td>
-                  <td style={{ padding: '6px 4px' }}>
-                    {t.category_ids.map(id => categoryNameById(id)).join(', ')}
-                  </td>
-                </tr>
-              ))}
-            </tbody>
-          </table>
-        )}
-      </section>
+          <section className="card">
+            <h3>Filters</h3>
+            <div className="form-row">
+              <input className="input" type="number" step="0.01" placeholder="Min amount" value={minAmount} onChange={(e) => setMinAmount(e.target.value)} />
+              <input className="input" type="number" step="0.01" placeholder="Max amount" value={maxAmount} onChange={(e) => setMaxAmount(e.target.value)} />
+              <select className="input" value={filterCategoryId} onChange={(e) => setFilterCategoryId(e.target.value ? Number(e.target.value) : '')}>
+                <option value="">All categories</option>
+                {categories.map(c => (<option key={c.id} value={c.id}>{c.name}</option>))}
+              </select>
+              <input className="input" type="text" placeholder="Search in description" value={searchText} onChange={(e) => setSearchText(e.target.value)} />
+            </div>
+          </section>
+
+          <section className="card">
+            <h3>Latest Transactions (last 10)</h3>
+            {loading ? (
+              <div>Loading…</div>
+            ) : error ? (
+              <div style={{ color: 'crimson' }}>{error}</div>
+            ) : filtered.length === 0 ? (
+              <div>No transactions</div>
+            ) : (
+              <table className="table">
+                <thead>
+                  <tr>
+                    <th>ID</th>
+                    <th style={{ textAlign: 'right' }}>Amount</th>
+                    <th>Description</th>
+                    <th>Categories</th>
+                  </tr>
+                </thead>
+                <tbody>
+                  {filtered.map(t => (
+                    <tr key={t.id}>
+                      <td>{t.id}</td>
+                      <td className="amount">{formatAmount(t.amount)}</td>
+                      <td>{t.description || ''}</td>
+                      <td>{t.category_ids.map(id => categoryNameById(id)).join(', ')}</td>
+                    </tr>
+                  ))}
+                </tbody>
+              </table>
+            )}
+          </section>
+        </main>
+      </div>
     </div>
   );
 }
diff --git a/7project/frontend/src/pages/LoginRegisterPage.tsx b/7project/frontend/src/pages/LoginRegisterPage.tsx
index e265167..ce333d0 100644
--- a/7project/frontend/src/pages/LoginRegisterPage.tsx
+++ b/7project/frontend/src/pages/LoginRegisterPage.tsx
@@ -1,6 +1,7 @@
-import { useState } from 'react';
+import { useState, useEffect } from 'react';
 import { login, register } from '../api';
 
+
 export default function LoginRegisterPage({ onLoggedIn }: { onLoggedIn: () => void }) {
   const [mode, setMode] = useState<'login' | 'register'>('login');
   const [email, setEmail] = useState('');
@@ -31,41 +32,54 @@ export default function LoginRegisterPage({ onLoggedIn }: { onLoggedIn: () => vo
     }
   }
 
+  // Add this useEffect hook
+  useEffect(() => {
+    // When the component mounts, add a class to the body
+    document.body.classList.add('auth-page');
+
+    // When the component unmounts, remove the class
+    return () => {
+      document.body.classList.remove('auth-page');
+    };
+  }, []); // The empty array ensures this runs only once
+
+  // The JSX no longer needs the wrapper div
   return (
-    <div style={{ maxWidth: 420, margin: '4rem auto', padding: 24, border: '1px solid #ddd', borderRadius: 8 }}>
-      <h2 style={{ marginTop: 0 }}>{mode === 'login' ? 'Login' : 'Register'}</h2>
-      <div style={{ marginBottom: 16 }}>
-        <button onClick={() => setMode('login')} disabled={mode === 'login'}>Login</button>
-        <button onClick={() => setMode('register')} disabled={mode === 'register'} style={{ marginLeft: 8 }}>Register</button>
+    <div className="card" style={{ width: 420 }}>
+      <div style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', marginBottom: 12 }}>
+        <h2 style={{ margin: 0 }}>{mode === 'login' ? 'Welcome back' : 'Create your account'}</h2>
+        <div className="segmented">
+          <button className={mode === 'login' ? 'active' : ''} type="button" onClick={() => setMode('login')}>Login</button>
+          <button className={mode === 'register' ? 'active' : ''} type="button" onClick={() => setMode('register')}>Register</button>
+        </div>
       </div>
-      <form onSubmit={handleSubmit}>
-        <div style={{ marginBottom: 12 }}>
-          <label>Email<br />
-            <input type="email" required value={email} onChange={(e) => setEmail(e.target.value)} style={{ width: '100%' }} />
-          </label>
-        </div>
-        <div style={{ marginBottom: 12 }}>
-          <label>Password<br />
-            <input type="password" required value={password} onChange={(e) => setPassword(e.target.value)} style={{ width: '100%' }} />
-          </label>
-        </div>
-        {mode === 'register' && (
-          <>
-            <div style={{ marginBottom: 12 }}>
-              <label>First name (optional)<br />
-                <input type="text" value={firstName} onChange={(e) => setFirstName(e.target.value)} style={{ width: '100%' }} />
-              </label>
+      <form onSubmit={handleSubmit} className="space-y">
+        <div>
+            <label className="muted">Email</label>
+            <input className="input" type="email" required value={email} onChange={(e) => setEmail(e.target.value)} />
+          </div>
+          <div>
+            <label className="muted">Password</label>
+            <input className="input" type="password" required value={password} onChange={(e) => setPassword(e.target.value)} />
+          </div>
+          {mode === 'register' && (
+            <div className="form-row">
+              <div>
+                <label className="muted">First name (optional)</label>
+                <input className="input" type="text" value={firstName} onChange={(e) => setFirstName(e.target.value)} />
+              </div>
+              <div>
+                <label className="muted">Last name (optional)</label>
+                <input className="input" type="text" value={lastName} onChange={(e) => setLastName(e.target.value)} />
+              </div>
             </div>
-            <div style={{ marginBottom: 12 }}>
-              <label>Last name (optional)<br />
-                <input type="text" value={lastName} onChange={(e) => setLastName(e.target.value)} style={{ width: '100%' }} />
-              </label>
-            </div>
-          </>
-        )}
-        {error && <div style={{ color: 'crimson', marginBottom: 12 }}>{error}</div>}
-        <button type="submit" disabled={loading}>{loading ? 'Please wait…' : (mode === 'login' ? 'Login' : 'Register')}</button>
+          )}
+          {error && <div style={{ color: 'crimson' }}>{error}</div>}
+          <div className="actions" style={{ justifyContent: 'flex-end' }}>
+            <button className="btn primary" type="submit" disabled={loading}>{loading ? 'Please wait…' : (mode === 'login' ? 'Login' : 'Register')}</button>
+          </div>
       </form>
     </div>
   );
 }
+