From bdf6fa6c2fd68cfe1d4f6a45fdc7b5d248558552 Mon Sep 17 00:00:00 2001
From: Marco Allegretti <mightymarco4@gmail.com>
Date: Thu, 29 Jan 2026 13:13:46 +0100
Subject: [PATCH] ui: refresh community proposals list

---
 .../communities/[slug]/proposals/index.astro  | 319 +++++++++++++-----
 1 file changed, 236 insertions(+), 83 deletions(-)

diff --git a/frontend/src/pages/communities/[slug]/proposals/index.astro b/frontend/src/pages/communities/[slug]/proposals/index.astro
index c2732ce..01865ea 100644
--- a/frontend/src/pages/communities/[slug]/proposals/index.astro
+++ b/frontend/src/pages/communities/[slug]/proposals/index.astro
@@ -6,28 +6,80 @@ const { slug } = Astro.params;
 ---
 
 <Layout title="Proposals">
-  <section class="proposals-page">
-    <div class="header-row">
-      <div>
-        <h1>Proposals</h1>
-        <p class="subtitle">Proposals and decisions</p>
-      </div>
-      <a href={`/communities/${slug}/proposals/new`} class="btn-create" id="create-btn" style="display: none;">+ New Proposal</a>
-    </div>
+  <section class="ui-page">
+    <div class="ui-container">
+      <div class="hero ui-card ui-card-glass">
+        <div class="hero-top">
+          <div>
+            <h1 class="hero-title">Proposals</h1>
+            <p class="hero-subtitle">Proposals and decisions</p>
+          </div>
+          <a href={`/communities/${slug}/proposals/new`} class="ui-btn ui-btn-primary" id="create-btn" style="display: none;">+ New Proposal</a>
+        </div>
 
-    <div class="filters">
-      <input type="text" id="search-input" placeholder="Search proposals..." />
-      <select id="status-filter">
-        <option value="">All Statuses</option>
-        <option value="draft">Draft</option>
-        <option value="discussion">Discussion</option>
-        <option value="voting">Voting</option>
-        <option value="closed">Closed</option>
-      </select>
-    </div>
-    
-    <div id="proposals-list" class="list">
-      <p class="loading">Loading proposals...</p>
+        <div class="hero-kpis">
+          <div class="hero-kpis-label">At a glance</div>
+          <div class="ui-kpis" id="kpis">
+            <div class="ui-kpi">
+              <div class="ui-kpi-value" id="kpi-total">—</div>
+              <div class="ui-kpi-label">Total</div>
+            </div>
+            <div class="ui-kpi">
+              <div class="ui-kpi-value" id="kpi-voting">—</div>
+              <div class="ui-kpi-label">Voting</div>
+            </div>
+            <div class="ui-kpi">
+              <div class="ui-kpi-value" id="kpi-discussion">—</div>
+              <div class="ui-kpi-label">Discussion</div>
+            </div>
+            <div class="ui-kpi">
+              <div class="ui-kpi-value" id="kpi-draft">—</div>
+              <div class="ui-kpi-label">Drafts</div>
+            </div>
+            <div class="ui-kpi">
+              <div class="ui-kpi-value" id="kpi-closed">—</div>
+              <div class="ui-kpi-label">Closed</div>
+            </div>
+          </div>
+        </div>
+      </div>
+
+      <div class="toolbar ui-card">
+        <div class="toolbar-grid">
+          <div>
+            <label class="toolbar-label" for="search-input">Search</label>
+            <input type="text" id="search-input" class="ui-input" placeholder="Search proposals…" />
+          </div>
+          <div>
+            <label class="toolbar-label" for="status-filter">Status</label>
+            <select id="status-filter" class="ui-select">
+              <option value="">All statuses</option>
+              <option value="draft">Draft</option>
+              <option value="discussion">Discussion</option>
+              <option value="voting">Voting</option>
+              <option value="closed">Closed</option>
+            </select>
+          </div>
+          <div>
+            <label class="toolbar-label" for="sort-select">Sort</label>
+            <select id="sort-select" class="ui-select">
+              <option value="newest">Newest</option>
+              <option value="oldest">Oldest</option>
+              <option value="status">Status</option>
+              <option value="title">Title</option>
+            </select>
+          </div>
+        </div>
+      </div>
+
+      <div id="proposals-list" class="list">
+        <div class="list-skeleton">
+          <div class="skeleton-card"></div>
+          <div class="skeleton-card"></div>
+          <div class="skeleton-card"></div>
+          <div class="skeleton-card"></div>
+        </div>
+      </div>
     </div>
   </section>
 </Layout>
@@ -35,6 +87,45 @@ const { slug } = Astro.params;
 <script define:vars={{ slug, apiBase }}>
   let allProposals = [];
 
+  function escapeHtml(value) {
+    return String(value || '').replace(/[&<>"']/g, function(ch) {
+      switch (ch) {
+        case '&': return '&amp;';
+        case '<': return '&lt;';
+        case '>': return '&gt;';
+        case '"': return '&quot;';
+        case "'": return '&#39;';
+        default: return ch;
+      }
+    });
+  }
+
+  function normalizeStatus(status) {
+    const s = String(status || '').toLowerCase();
+    if (s === 'draft' || s === 'discussion' || s === 'voting' || s === 'closed') return s;
+    return 'draft';
+  }
+
+  function updateKpis(proposals) {
+    const total = proposals.length;
+    const byStatus = { draft: 0, discussion: 0, voting: 0, closed: 0 };
+    proposals.forEach((p) => {
+      const key = normalizeStatus(p.status);
+      byStatus[key] = (byStatus[key] || 0) + 1;
+    });
+
+    const setText = (id, value) => {
+      const el = document.getElementById(id);
+      if (el) el.textContent = String(value);
+    };
+
+    setText('kpi-total', total);
+    setText('kpi-draft', byStatus.draft);
+    setText('kpi-discussion', byStatus.discussion);
+    setText('kpi-voting', byStatus.voting);
+    setText('kpi-closed', byStatus.closed);
+  }
+
   async function loadProposals() {
     const container = document.getElementById('proposals-list');
     if (!container) return;
@@ -51,9 +142,10 @@ const { slug } = Astro.params;
 
       const res = await fetch(`${apiBase}/api/communities/${community.id}/proposals`);
       allProposals = await res.json();
-      renderProposals(allProposals);
+      updateKpis(allProposals);
+      filterProposals();
     } catch (error) {
-      container.innerHTML = '<div class="error"><p>Failed to load proposals.</p></div>';
+      container.innerHTML = '<div class="ui-card"><p class="error">Failed to load proposals.</p></div>';
     }
   }
 
@@ -62,27 +154,39 @@ const { slug } = Astro.params;
     if (!container) return;
 
     if (proposals.length === 0) {
-      container.innerHTML = '<div class="empty"><p>No proposals found.</p></div>';
+      container.innerHTML = '<div class="ui-card"><p class="empty">No proposals found.</p></div>';
       return;
     }
 
-    container.innerHTML = proposals.map(p => `
-      <a href="/proposals/${p.id}" class="proposal-card">
-        <div class="proposal-header">
-          <h3>${p.title}</h3>
-          <span class="status status-${p.status}">${p.status}</span>
-        </div>
-        <p class="description">${p.description.substring(0, 150)}${p.description.length > 150 ? '...' : ''}</p>
-      </a>
-    `).join('');
+    container.innerHTML = proposals.map(p => {
+      const statusKey = normalizeStatus(p.status);
+      const safeTitle = escapeHtml(p.title);
+      const desc = String(p.description || '');
+      const safeDesc = escapeHtml(desc.substring(0, 160));
+      const suffix = desc.length > 160 ? '…' : '';
+      const createdAt = p.created_at ? new Date(p.created_at).toLocaleDateString() : '';
+
+      return `
+        <a href="/proposals/${encodeURIComponent(String(p.id))}" class="proposal-card ui-card">
+          <div class="proposal-header">
+            <h3>${safeTitle}</h3>
+            <span class="status status-${statusKey}">${escapeHtml(statusKey)}</span>
+          </div>
+          ${createdAt ? `<div class="proposal-meta">Created ${escapeHtml(createdAt)}</div>` : ''}
+          <p class="description">${safeDesc}${suffix}</p>
+        </a>
+      `;
+    }).join('');
   }
 
   function filterProposals() {
     const searchInput = document.getElementById('search-input');
     const statusFilter = document.getElementById('status-filter');
+    const sortSelect = document.getElementById('sort-select');
     
     const query = searchInput?.value.toLowerCase().trim() || '';
     const status = statusFilter?.value || '';
+    const sortKey = sortSelect?.value || 'newest';
 
     let filtered = allProposals;
 
@@ -97,6 +201,26 @@ const { slug } = Astro.params;
       filtered = filtered.filter(p => p.status === status);
     }
 
+    const getTime = (p) => {
+      const d = p.created_at ? Date.parse(p.created_at) : NaN;
+      if (!Number.isNaN(d)) return d;
+      const asNum = Number(p.id);
+      if (!Number.isNaN(asNum)) return asNum;
+      return 0;
+    };
+
+    filtered = [...filtered];
+    if (sortKey === 'newest') {
+      filtered.sort((a, b) => getTime(b) - getTime(a));
+    } else if (sortKey === 'oldest') {
+      filtered.sort((a, b) => getTime(a) - getTime(b));
+    } else if (sortKey === 'title') {
+      filtered.sort((a, b) => String(a.title || '').localeCompare(String(b.title || '')));
+    } else if (sortKey === 'status') {
+      const order = { voting: 0, discussion: 1, draft: 2, closed: 3 };
+      filtered.sort((a, b) => (order[normalizeStatus(a.status)] ?? 99) - (order[normalizeStatus(b.status)] ?? 99));
+    }
+
     renderProposals(filtered);
   }
 
@@ -104,75 +228,77 @@ const { slug } = Astro.params;
 
   document.getElementById('search-input')?.addEventListener('input', filterProposals);
   document.getElementById('status-filter')?.addEventListener('change', filterProposals);
+  document.getElementById('sort-select')?.addEventListener('change', filterProposals);
 
   const token = localStorage.getItem('token');
   const createBtn = document.getElementById('create-btn');
   if (token && createBtn) {
-    createBtn.style.display = 'block';
+    createBtn.style.display = 'inline-flex';
   }
 </script>
 
 <style>
-  .proposals-page {
-    padding: 2rem 0;
+  .hero {
+    padding: 1.25rem;
+    margin-bottom: 1rem;
   }
 
-  .header-row {
+  .hero-top {
     display: flex;
     justify-content: space-between;
     align-items: flex-start;
-    margin-bottom: 2rem;
+    gap: 1rem;
+    flex-wrap: wrap;
   }
 
-  h1 {
-    font-size: 2.5rem;
-    margin-bottom: 0.5rem;
+  .hero-title {
+    margin: 0;
+    font-size: 2.125rem;
+    letter-spacing: -0.02em;
   }
 
-  .subtitle {
+  .hero-subtitle {
+    margin: 0.25rem 0 0;
     color: var(--color-text-muted);
   }
 
-  .btn-create {
-    background: var(--color-primary);
-    color: var(--color-on-primary);
-    padding: 0.75rem 1.5rem;
-    border-radius: 8px;
-    font-weight: 600;
+  .hero-kpis {
+    margin-top: 1rem;
   }
 
-  .btn-create:hover {
-    background: var(--color-primary-hover);
-    color: var(--color-on-primary);
+  .hero-kpis-label {
+    color: var(--color-text-muted);
+    font-size: 0.8125rem;
+    margin-bottom: 0.5rem;
+    letter-spacing: 0.02em;
+    text-transform: uppercase;
   }
 
-  .filters {
-    display: flex;
-    gap: 1rem;
-    margin-bottom: 1.5rem;
+  .toolbar {
+    padding: 1rem;
+    margin-bottom: 1rem;
   }
 
-  .filters input, .filters select {
-    padding: 0.75rem 1rem;
-    border: 1px solid var(--color-border);
-    border-radius: 8px;
-    background: var(--color-surface);
-    color: var(--color-text);
-    font-size: 1rem;
+  .toolbar-grid {
+    display: grid;
+    grid-template-columns: 1.5fr 1fr 1fr;
+    gap: 0.75rem;
+    align-items: end;
   }
 
-  .filters input {
-    flex: 1;
-    max-width: 300px;
+  .toolbar-label {
+    display: block;
+    font-size: 0.75rem;
+    letter-spacing: 0.02em;
+    text-transform: uppercase;
+    color: var(--color-text-muted);
+    margin-bottom: 0.35rem;
   }
 
-  .filters select {
-    min-width: 150px;
-  }
-
-  .filters input:focus, .filters select:focus {
-    outline: none;
-    border-color: var(--color-primary);
+  @media (max-width: 768px) {
+    .toolbar-grid {
+      grid-template-columns: 1fr;
+    }
   }
 
   .list {
@@ -182,16 +308,15 @@ const { slug } = Astro.params;
 
   .proposal-card {
     display: block;
-    background: var(--color-surface);
-    border: 1px solid var(--color-border);
-    border-radius: 12px;
-    padding: 1.5rem;
+    padding: 1.1rem;
     color: var(--color-text);
-    transition: border-color 0.2s;
+    transition: transform 140ms ease, border-color 140ms ease, background 140ms ease;
   }
 
   .proposal-card:hover {
-    border-color: var(--color-primary);
+    border-color: var(--color-border-hover);
+    transform: translateY(-2px);
+    background: rgba(255, 255, 255, 0.03);
   }
 
   .proposal-header {
@@ -199,10 +324,18 @@ const { slug } = Astro.params;
     justify-content: space-between;
     align-items: center;
     margin-bottom: 0.5rem;
+    gap: 0.75rem;
   }
 
   .proposal-card h3 {
     font-size: 1.25rem;
+    letter-spacing: -0.01em;
+  }
+
+  .proposal-meta {
+    color: var(--color-text-muted);
+    font-size: 0.8125rem;
+    margin-bottom: 0.5rem;
   }
 
   .status {
@@ -221,16 +354,36 @@ const { slug } = Astro.params;
   .description {
     color: var(--color-text-muted);
     font-size: 0.875rem;
+    line-height: 1.5;
   }
 
-  .loading, .empty, .error {
+  .empty, .error {
     text-align: center;
-    padding: 3rem;
+    padding: 2rem;
     color: var(--color-text-muted);
   }
 
-  .hint {
-    font-size: 0.875rem;
-    margin-top: 0.5rem;
+  .list-skeleton {
+    display: grid;
+    gap: 1rem;
+  }
+
+  .skeleton-card {
+    height: 92px;
+    border-radius: 14px;
+    border: 1px solid var(--color-border);
+    background: linear-gradient(
+      90deg,
+      rgba(255, 255, 255, 0.03) 0%,
+      rgba(255, 255, 255, 0.06) 50%,
+      rgba(255, 255, 255, 0.03) 100%
+    );
+    background-size: 200% 100%;
+    animation: shimmer 1.2s ease-in-out infinite;
+  }
+
+  @keyframes shimmer {
+    0% { background-position: 0% 0%; }
+    100% { background-position: 200% 0%; }
   }
 </style>