From 2dbc43a4ebc37e998fc65a9a4a0df4bc1b07c896 Mon Sep 17 00:00:00 2001
From: Dwindi Ramadhana <dwindown@DwMacMiniM4.local>
Date: Wed, 31 Dec 2025 14:27:06 +0700
Subject: [PATCH] fix: simplify More page - Marketing as simple button without
 submenu

- Remove inline submenu expansion for Marketing
- Keep it consistent with Appearance and Settings (simple buttons)
- Description provides enough context about what's inside
---
 admin-spa/src/routes/More/index.tsx | 52 +++++++++--------------------
 1 file changed, 16 insertions(+), 36 deletions(-)

diff --git a/admin-spa/src/routes/More/index.tsx b/admin-spa/src/routes/More/index.tsx
index abed9e6..d08f863 100644
--- a/admin-spa/src/routes/More/index.tsx
+++ b/admin-spa/src/routes/More/index.tsx
@@ -12,7 +12,6 @@ interface MenuItem {
   label: string;
   description: string;
   to: string;
-  children?: { label: string; to: string }[];
 }
 
 const menuItems: MenuItem[] = [
@@ -20,11 +19,7 @@ const menuItems: MenuItem[] = [
     icon: <Megaphone className="w-5 h-5" />,
     label: __('Marketing'),
     description: __('Newsletter, coupons, and promotions'),
-    to: '/marketing',
-    children: [
-      { label: __('Newsletter'), to: '/marketing/newsletter' },
-      { label: __('Coupons'), to: '/coupons' },
-    ]
+    to: '/marketing'
   },
   {
     icon: <Palette className="w-5 h-5" />,
@@ -80,37 +75,22 @@ export default function MorePage() {
       {/* Menu Items */}
       <div className="divide-y">
         {menuItems.map((item) => (
-          <div key={item.to}>
-            <button
-              onClick={() => navigate(item.to)}
-              className="w-full flex items-center gap-4 py-4 hover:bg-accent transition-colors"
-            >
-              <div className="flex-shrink-0 w-10 h-10 rounded-lg bg-primary/10 text-primary flex items-center justify-center">
-                {item.icon}
+          <button
+            key={item.to}
+            onClick={() => navigate(item.to)}
+            className="w-full flex items-center gap-4 py-4 hover:bg-accent transition-colors"
+          >
+            <div className="flex-shrink-0 w-10 h-10 rounded-lg bg-primary/10 text-primary flex items-center justify-center">
+              {item.icon}
+            </div>
+            <div className="flex-1 text-left min-w-0">
+              <div className="font-medium">{item.label}</div>
+              <div className="text-sm text-muted-foreground truncate">
+                {item.description}
               </div>
-              <div className="flex-1 text-left min-w-0">
-                <div className="font-medium">{item.label}</div>
-                <div className="text-sm text-muted-foreground truncate">
-                  {item.description}
-                </div>
-              </div>
-              <ChevronRight className="w-5 h-5 text-muted-foreground flex-shrink-0" />
-            </button>
-            {/* Submenu items */}
-            {item.children && item.children.length > 0 && (
-              <div className="pl-14 pb-2 space-y-1">
-                {item.children.map((child) => (
-                  <button
-                    key={child.to}
-                    onClick={() => navigate(child.to)}
-                    className="w-full text-left py-2 px-3 text-sm text-muted-foreground hover:text-foreground hover:bg-accent rounded-md transition-colors"
-                  >
-                    {child.label}
-                  </button>
-                ))}
-              </div>
-            )}
-          </div>
+            </div>
+            <ChevronRight className="w-5 h-5 text-muted-foreground flex-shrink-0" />
+          </button>
         ))}
       </div>