From 7eafd45ab1ae2246a49a980601e9ddf9cf06c2b1 Mon Sep 17 00:00:00 2001
From: Daniel Bedeleanu <bede@tfm.ro>
Date: Sun, 19 Apr 2026 18:45:14 +0300
Subject: [PATCH] refactor: polish spacing details (Phase 3 - Low-Impact)

---
 frontend/components/admin/AiManager.tsx       |  4 ++--
 frontend/components/admin/CategoryManager.tsx |  6 +++---
 frontend/components/admin/DatabaseManager.tsx |  2 +-
 frontend/components/admin/IdentityManager.tsx |  4 ++--
 frontend/components/admin/LdapManager.tsx     | 20 +++++++++----------
 5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/frontend/components/admin/AiManager.tsx b/frontend/components/admin/AiManager.tsx
index b7e87e35..a1d599d5 100644
--- a/frontend/components/admin/AiManager.tsx
+++ b/frontend/components/admin/AiManager.tsx
@@ -33,7 +33,7 @@ export default function AiManager({
 }: AiManagerProps) {
   return (
     <section data-testid="ai-config" className="bg-surface/50 border border-slate-800/50 rounded-[2.5rem] p-5 md:p-8 shadow-2xl space-y-6 transition-all group/ai">
-      <div className="flex items-center justify-between mb-2">
+      <div className="flex items-center justify-between mb-1">
         <div className="flex items-center gap-4">
           <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
             <Brain size={20} />
@@ -156,7 +156,7 @@ export default function AiManager({
         </div>
       </div>
 
-      <div className="space-y-3">
+      <div className="space-y-3 pt-1">
         <div className="space-y-1.5">
           <div className="flex items-center justify-between px-1">
             <label className="text-sm font-normal text-secondary tracking-tight">System Prompt (Vision Extraction)</label>
diff --git a/frontend/components/admin/CategoryManager.tsx b/frontend/components/admin/CategoryManager.tsx
index 84f49bff..d8232474 100644
--- a/frontend/components/admin/CategoryManager.tsx
+++ b/frontend/components/admin/CategoryManager.tsx
@@ -24,7 +24,7 @@ export default function CategoryManager({
 }: CategoryManagerProps) {
   return (
     <section className="bg-surface/50 border border-slate-800/50 rounded-[2.5rem] p-5 md:p-8 shadow-2xl space-y-6 transition-all group/categories">
-      <div className="flex items-center justify-between px-2">
+      <div className="flex items-center justify-between px-2 mb-2">
          <div className="flex items-center gap-4">
            <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
              <Layers size={20} />
@@ -72,8 +72,8 @@ export default function CategoryManager({
       {/* Edit Category Modal */}
       {editingCategory && (
         <div className="fixed inset-0 z-[100] flex items-end sm:items-center justify-center p-0 sm:p-4 bg-background/90 animate-in fade-in duration-300">
-          <div className="w-full max-w-lg bg-surface border-x border-t sm:border border-slate-800 rounded-t-[2.5rem] sm:rounded-3xl p-6 sm:p-10 shadow-2xl space-y-8 overflow-hidden animate-in slide-in-from-bottom-10">
-            <div className="flex justify-between items-center">
+          <div className="w-full max-w-lg bg-surface border-x border-t sm:border border-slate-800 rounded-t-[2.5rem] sm:rounded-3xl p-6 sm:p-10 shadow-2xl space-y-4 overflow-hidden animate-in slide-in-from-bottom-10">
+            <div className="flex justify-between items-center mb-2">
                <div className="flex items-center gap-4">
                  <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
                    <Edit2 size={20} />
diff --git a/frontend/components/admin/DatabaseManager.tsx b/frontend/components/admin/DatabaseManager.tsx
index 3e2dabfa..d94e4a30 100644
--- a/frontend/components/admin/DatabaseManager.tsx
+++ b/frontend/components/admin/DatabaseManager.tsx
@@ -36,7 +36,7 @@ export default function DatabaseManager({
   return (
     <div className="space-y-4 md:space-y-5 h-full">
       <div data-testid="database-info" className="bg-surface/50 border border-slate-800/50 rounded-[2.5rem] p-4 md:p-6 shadow-2xl overflow-hidden relative group transition-all">
-        <div className="flex items-center gap-4 mb-6">
+        <div className="flex items-center gap-4 mb-4">
           <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
             <Database size={20} />
           </div>
diff --git a/frontend/components/admin/IdentityManager.tsx b/frontend/components/admin/IdentityManager.tsx
index 487ad9ec..557d1a20 100644
--- a/frontend/components/admin/IdentityManager.tsx
+++ b/frontend/components/admin/IdentityManager.tsx
@@ -25,7 +25,7 @@ export default function IdentityManager({
 }: IdentityManagerProps) {
   return (
     <div className="bg-surface/50  border border-slate-800/50 rounded-[2.5rem] p-4 md:p-6 flex flex-col shadow-2xl transition-all group/identity h-full">
-      <div className="flex items-center justify-between mb-6">
+      <div className="flex items-center justify-between mb-4">
         <div className="flex items-center gap-4">
           <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
             <User size={20} />
@@ -87,7 +87,7 @@ export default function IdentityManager({
       {editingUser && (
         <div className="fixed inset-0 z-[100] flex items-center justify-center p-4 bg-background/80  animate-in fade-in duration-200">
           <div className="bg-surface border border-slate-800 rounded-[2.5rem] p-6 w-full max-w-md shadow-2xl">
-            <div className="flex justify-between items-center mb-4">
+            <div className="flex justify-between items-center mb-2">
               <h3 className="text-xl font-normal text-white tracking-tight">Edit Identity</h3>
               <button
                 onClick={() => setEditingUser(null)}
diff --git a/frontend/components/admin/LdapManager.tsx b/frontend/components/admin/LdapManager.tsx
index 5a603c6b..c6550586 100644
--- a/frontend/components/admin/LdapManager.tsx
+++ b/frontend/components/admin/LdapManager.tsx
@@ -19,7 +19,7 @@ export default function LdapManager({
 }: LdapManagerProps) {
   return (
     <div className="bg-surface/50 border border-slate-800/50 rounded-[2.5rem] p-4 md:p-6 shadow-2xl space-y-4 transition-all group/ldap">
-      <div className="flex items-center justify-between mb-2">
+      <div className="flex items-center justify-between mb-1">
         <div className="flex items-center gap-4">
           <div className="w-10 h-10 rounded-xl bg-primary/10 flex items-center justify-center text-primary border border-primary/20">
             <Globe size={20} />
@@ -61,26 +61,26 @@ export default function LdapManager({
         <div className="space-y-1">
           <label className="text-sm font-normal text-secondary tracking-tight ml-1">LDAP URI</label>
           <div className="relative">
-            <Server className="absolute left-3.5 top-1/2 -translate-y-1/2 text-muted" size={14} />
+            <Server className="absolute left-3 top-1/2 -translate-y-1/2 text-muted" size={14} />
             <input
               type="text"
               placeholder="ldap://host:389"
               value={ldapConfig.server_uri}
               onChange={(e) => setLdapConfig({...ldapConfig, server_uri: e.target.value})}
-              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-10 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
+              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-9 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
             />
           </div>
         </div>
         <div className="space-y-1">
           <label className="text-sm font-normal text-secondary tracking-tight ml-1">Context DN</label>
           <div className="relative">
-            <Shield className="absolute left-3.5 top-1/2 -translate-y-1/2 text-muted" size={14} />
+            <Shield className="absolute left-3 top-1/2 -translate-y-1/2 text-muted" size={14} />
             <input
               type="text"
               placeholder="dc=example,dc=com"
               value={ldapConfig.base_dn}
               onChange={(e) => setLdapConfig({...ldapConfig, base_dn: e.target.value})}
-              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-10 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
+              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-9 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
             />
           </div>
         </div>
@@ -90,26 +90,26 @@ export default function LdapManager({
         <div className="space-y-1">
           <label className="text-sm font-normal text-secondary tracking-tight ml-1">User DN Template</label>
           <div className="relative">
-            <User className="absolute left-3.5 top-1/2 -translate-y-1/2 text-muted" size={14} />
+            <User className="absolute left-3 top-1/2 -translate-y-1/2 text-muted" size={14} />
             <input
               type="text"
               placeholder="uid={username},ou=people..."
               value={ldapConfig.user_template}
               onChange={(e) => setLdapConfig({...ldapConfig, user_template: e.target.value})}
-              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-10 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
+              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-9 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
             />
           </div>
         </div>
         <div className="space-y-1">
           <label className="text-sm font-normal text-secondary tracking-tight ml-1">Groups Base DN</label>
           <div className="relative">
-            <Layers className="absolute left-3.5 top-1/2 -translate-y-1/2 text-muted" size={14} />
+            <Layers className="absolute left-3 top-1/2 -translate-y-1/2 text-muted" size={14} />
             <input
               type="text"
               placeholder="ou=groups"
               value={ldapConfig.groups_dn}
               onChange={(e) => setLdapConfig({...ldapConfig, groups_dn: e.target.value})}
-              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-10 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
+              className="w-full bg-background/80 border border-slate-800 rounded-2xl py-2 pl-9 pr-4 text-xs font-normal text-white outline-none focus:border-primary/50 transition-all font-mono"
             />
           </div>
         </div>
@@ -144,7 +144,7 @@ export default function LdapManager({
         </button>
       </div>
 
-      <div className="flex gap-2 pt-2">
+      <div className="flex gap-2 pt-1">
         <button
           onClick={onTestLdap}
           disabled={testingLdap}