From 3b5b0d831d75c803ee479eb74dd5b73f668b48e1 Mon Sep 17 00:00:00 2001
From: Jennie Robinson Faber <jennie@jenniefaber.com>
Date: Wed, 8 Apr 2026 17:58:01 +0100
Subject: [PATCH] =?UTF-8?q?refactor:=20phase=204=20cleanup=20=E2=80=94=20d?=
 =?UTF-8?q?elete=20SidebarLayout,=20drop=20dashboard-body?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Now that all member-area pages have migrated to PageShell +
ColumnsLayout, SidebarLayout has no consumers and can be deleted.
PageShell owns the flex chain, so the .dashboard-body wrapper on
member/dashboard.vue (flex: 1; display: flex; flex-direction: column;
min-height: 0) is redundant. Update stale SidebarLayout comments on
members/[id].vue to reference ColumnsLayout.
---
 app/components/SidebarLayout.vue | 43 --------------------------------
 app/pages/member/dashboard.vue   | 13 ++--------
 app/pages/members/[id].vue       |  6 ++---
 3 files changed, 5 insertions(+), 57 deletions(-)
 delete mode 100644 app/components/SidebarLayout.vue

diff --git a/app/components/SidebarLayout.vue b/app/components/SidebarLayout.vue
deleted file mode 100644
index dbae39c..0000000
--- a/app/components/SidebarLayout.vue
+++ /dev/null
@@ -1,43 +0,0 @@
-<template>
-  <div class="sidebar-layout">
-    <div class="sidebar-layout-main">
-      <slot />
-    </div>
-    <EventsMiniSidebar :events="upcomingEvents" />
-  </div>
-</template>
-
-<script setup>
-const props = defineProps({
-  limit: { type: Number, default: 3 },
-})
-
-const { data: upcomingEvents } = await useFetch('/api/events', {
-  query: { limit: props.limit, upcoming: true },
-  default: () => [],
-})
-</script>
-
-<style scoped>
-.sidebar-layout {
-  flex: 1;
-  display: grid;
-  grid-template-columns: 1fr 200px;
-  align-items: stretch;
-  min-height: 0;
-}
-
-.sidebar-layout-main {
-  min-width: 0;
-  align-self: stretch;
-  display: flex;
-  flex-direction: column;
-  min-height: 0;
-}
-
-@media (max-width: 1024px) {
-  .sidebar-layout {
-    grid-template-columns: 1fr;
-  }
-}
-</style>
diff --git a/app/pages/member/dashboard.vue b/app/pages/member/dashboard.vue
index e08fb80..de90afa 100644
--- a/app/pages/member/dashboard.vue
+++ b/app/pages/member/dashboard.vue
@@ -27,9 +27,8 @@
       <!-- Dashboard Content -->
       <template v-else>
         <ColumnsLayout cols="events-sidebar" :limit="5">
-          <div class="dashboard-body">
-            <!-- Member Status Banner -->
-            <MemberStatusBanner />
+          <!-- Member Status Banner -->
+          <MemberStatusBanner />
 
             <!-- Welcome Header -->
             <PageHeader :title="`Welcome back, ${memberData?.name || ''}`">
@@ -202,7 +201,6 @@
                 </DashedBox>
               </div>
             </div>
-          </div>
         </ColumnsLayout>
       </template>
 
@@ -451,13 +449,6 @@ useHead({
   margin-top: 8px;
 }
 
-.dashboard-body {
-  flex: 1;
-  display: flex;
-  flex-direction: column;
-  min-height: 0;
-}
-
 .content-row {
   display: grid;
   grid-template-columns: repeat(2, minmax(0, 1fr));
diff --git a/app/pages/members/[id].vue b/app/pages/members/[id].vue
index 00011f8..3716b9c 100644
--- a/app/pages/members/[id].vue
+++ b/app/pages/members/[id].vue
@@ -15,7 +15,7 @@
     <!-- Profile Content -->
     <template v-else>
 
-      <!-- HERO: full-bleed, outside SidebarLayout -->
+      <!-- HERO: full-bleed, outside ColumnsLayout -->
       <div class="profile-hero" :class="{ 'profile-hero--with-links': hasSocialLinks }">
 
         <!-- Left: Avatar + Identity -->
@@ -514,7 +514,7 @@ useHead({
 }
 
 /* ====================================================
-   SECTIONS — inside SidebarLayout
+   SECTIONS — inside ColumnsLayout
    ==================================================== */
 
 .profile-section {
@@ -734,7 +734,7 @@ useHead({
    ==================================================== */
 
 @media (max-width: 1024px) {
-  /* SidebarLayout sidebar hides itself at ≤1024px */
+  /* ColumnsLayout events-sidebar hides itself at ≤1024px */
   .profile-two-col {
     grid-template-columns: 1fr;
   }