From 92a9011f270e0a8f66570173ea0a6203cb34d952 Mon Sep 17 00:00:00 2001
From: filipriec <filippriec@gmail.com>
Date: Fri, 18 Apr 2025 11:04:13 +0200
Subject: [PATCH] buttons are only border and text colors now in add_table

---
 client/src/components/admin/add_table.rs | 80 ++++++++----------------
 1 file changed, 26 insertions(+), 54 deletions(-)

diff --git a/client/src/components/admin/add_table.rs b/client/src/components/admin/add_table.rs
index 8df8a96..db6a26b 100644
--- a/client/src/components/admin/add_table.rs
+++ b/client/src/components/admin/add_table.rs
@@ -248,22 +248,22 @@ pub fn render_add_table(
 
     // --- Button Style Helpers ---
     let get_button_style = |button_focus: AddTableFocus, current_focus| {
+        // Only handles text style (FG + Bold) now, no BG
         let is_focused = current_focus == button_focus;
         let base_style = Style::default().fg(if is_focused {
-            theme.bg // Reversed text color
+            theme.highlight // Highlighted text color
         } else {
             theme.secondary // Normal text color
         });
         if is_focused {
-            base_style
-                .add_modifier(Modifier::BOLD)
-                .bg(theme.highlight) // Reversed background
+            base_style.add_modifier(Modifier::BOLD)
         } else {
             base_style
         }
     };
-    let get_button_border_style = |button_focus: AddTableFocus, current_focus| {
-        if current_focus == button_focus {
+    // Updated signature to accept bool and theme
+    let get_button_border_style = |is_focused: bool, theme: &Theme| {
+        if is_focused {
             Style::default().fg(theme.highlight)
         } else {
             Style::default().fg(theme.secondary)
@@ -271,50 +271,22 @@ pub fn render_add_table(
     };
 
     // --- Add Button Rendering ---
-    let is_add_button_focused =
-        add_table_state.current_focus == AddTableFocus::AddColumnButton;
+    // Determine if the add button is focused
+    let is_add_button_focused = add_table_state.current_focus == AddTableFocus::AddColumnButton;
 
-    // 1. Define the block with ONLY border styling (no background here)
-    let add_button_block = Block::default()
-        .borders(Borders::ALL)
-        .border_type(BorderType::Rounded)
-        .border_style(if is_add_button_focused {
-            Style::default().fg(theme.highlight) // Highlighted border
-        } else {
-            Style::default().fg(theme.secondary) // Normal border
-        });
-    // DO NOT add .style(Style::default().bg(...)) here
-
-    // 2. Render the border block first
-    //    Need to clone the block because inner() consumes it
-    let inner_add_button_area = add_button_block.clone().inner(add_button_area);
-    f.render_widget(add_button_block, add_button_area);
-
-    // 3. If focused, render a background fill widget INSIDE the inner area
-    if is_add_button_focused {
-        let background_fill = Block::default() // Or Paragraph::new("")
-            .style(Style::default().bg(theme.highlight));
-        f.render_widget(background_fill, inner_add_button_area);
-    }
-
-    // 4. Define the paragraph text style (FG color and bold only)
-    let mut add_button_text_style = Style::default().fg(if is_add_button_focused {
-        theme.bg // Reversed text color for contrast on highlight bg
-    } else {
-        theme.secondary // Normal text color
-    });
-
-    if is_add_button_focused {
-        add_button_text_style = add_button_text_style.add_modifier(Modifier::BOLD);
-    }
-
-    let add_button_paragraph = Paragraph::new(" Add ")
-        .style(add_button_text_style) // Style without BG
-        .alignment(Alignment::Center);
-
-    // 5. Render the text paragraph inside the inner area (on top of the background fill)
-    f.render_widget(add_button_paragraph, inner_add_button_area);
+    // Create the Add button Paragraph widget
+    let add_button = Paragraph::new(" Add ")
+        .style(get_button_style(AddTableFocus::AddColumnButton, add_table_state.current_focus)) // Use existing closure
+        .alignment(Alignment::Center)
+        .block(
+            Block::default()
+            .borders(Borders::ALL)
+            .border_type(BorderType::Rounded)
+            .border_style(get_button_border_style(is_add_button_focused, theme)), // Pass bool and theme
+        );
 
+    // Render the button in its designated area
+    f.render_widget(add_button, add_button_area);
 
     // --- Indexes Table Rendering ---
     let indexes_focused = matches!(add_table_state.current_focus, AddTableFocus::IndexesTable | AddTableFocus::InsideIndexesTable);
@@ -426,8 +398,8 @@ pub fn render_add_table(
                 .borders(Borders::ALL)
                 .border_type(BorderType::Rounded)
                 .border_style(get_button_border_style(
-                    AddTableFocus::SaveButton,
-                    add_table_state.current_focus,
+                    add_table_state.current_focus == AddTableFocus::SaveButton, // Pass bool
+                    theme,
                 )),
         );
     f.render_widget(save_button, bottom_button_chunks[0]);
@@ -443,8 +415,8 @@ pub fn render_add_table(
                 .borders(Borders::ALL)
                 .border_type(BorderType::Rounded)
                 .border_style(get_button_border_style(
-                    AddTableFocus::DeleteSelectedButton,
-                    add_table_state.current_focus,
+                    add_table_state.current_focus == AddTableFocus::DeleteSelectedButton, // Pass bool
+                    theme,
                 )),
         );
     f.render_widget(delete_button, bottom_button_chunks[1]);
@@ -460,8 +432,8 @@ pub fn render_add_table(
                 .borders(Borders::ALL)
                 .border_type(BorderType::Rounded)
                 .border_style(get_button_border_style(
-                    AddTableFocus::CancelButton,
-                    add_table_state.current_focus,
+                    add_table_state.current_focus == AddTableFocus::CancelButton, // Pass bool
+                    theme,
                 )),
         );
     f.render_widget(cancel_button, bottom_button_chunks[2]);