From 539c204bde298159d69bf3636d3c52fa5a689bde Mon Sep 17 00:00:00 2001
From: fahed <fahed@softhouse.io>
Date: Mon, 9 Mar 2026 14:28:38 +0300
Subject: [PATCH] fix: add delete button inside brand edit modal

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 client/src/pages/Brands.jsx | 38 +++++++++++++++++++++++--------------
 1 file changed, 24 insertions(+), 14 deletions(-)

diff --git a/client/src/pages/Brands.jsx b/client/src/pages/Brands.jsx
index 961811c..9bf0947 100644
--- a/client/src/pages/Brands.jsx
+++ b/client/src/pages/Brands.jsx
@@ -297,20 +297,30 @@ export default function Brands() {
             </div>
           )}
 
-          <div className="flex items-center justify-end gap-3 pt-4 border-t border-border">
-            <button
-              onClick={() => { setShowModal(false); setEditingBrand(null) }}
-              className="px-4 py-2 text-sm font-medium text-text-secondary hover:bg-surface-tertiary rounded-lg"
-            >
-              {t('common.cancel')}
-            </button>
-            <button
-              onClick={saveBrand}
-              disabled={!brandForm.name}
-              className="px-5 py-2 bg-brand-primary text-white rounded-lg text-sm font-medium hover:bg-brand-primary-light disabled:opacity-50 disabled:cursor-not-allowed shadow-sm"
-            >
-              {t('common.save')}
-            </button>
+          <div className="flex items-center justify-between pt-4 border-t border-border">
+            {editingBrand && isSuperadminOrManager ? (
+              <button
+                onClick={() => { setShowModal(false); setBrandToDelete(editingBrand); setShowDeleteModal(true) }}
+                className="px-3 py-2 text-sm font-medium text-red-600 hover:bg-red-50 rounded-lg transition-colors"
+              >
+                {t('common.delete')}
+              </button>
+            ) : <div />}
+            <div className="flex items-center gap-3">
+              <button
+                onClick={() => { setShowModal(false); setEditingBrand(null) }}
+                className="px-4 py-2 text-sm font-medium text-text-secondary hover:bg-surface-tertiary rounded-lg"
+              >
+                {t('common.cancel')}
+              </button>
+              <button
+                onClick={saveBrand}
+                disabled={!brandForm.name}
+                className="px-5 py-2 bg-brand-primary text-white rounded-lg text-sm font-medium hover:bg-brand-primary-light disabled:opacity-50 disabled:cursor-not-allowed shadow-sm"
+              >
+                {t('common.save')}
+              </button>
+            </div>
           </div>
         </div>
       </Modal>