From 496ad4d1dc17fc356c2eb1e31976e57dc0145969 Mon Sep 17 00:00:00 2001
From: Erik Darling <2136037+erikdarlingdata@users.noreply.github.com>
Date: Wed, 18 Feb 2026 19:55:54 -0500
Subject: [PATCH] Make Lite filter-active indicator more obvious
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When a column filter is active:
- Icon changes from outline filter (E71C) to filled filter (E16E)
- Gold-tinted semi-transparent background added behind the icon
- Hover state brightens both icon and background
- Tooltip says "Filter active — click to modify"

The DataGridFilterManager also updated to use the filled icon when
dynamically creating filter button content.

Partial fix for #110

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 Lite/Services/DataGridFilterManager.cs | 2 +-
 Lite/Themes/DarkTheme.xaml             | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/Lite/Services/DataGridFilterManager.cs b/Lite/Services/DataGridFilterManager.cs
index 37026a3e..cf088387 100644
--- a/Lite/Services/DataGridFilterManager.cs
+++ b/Lite/Services/DataGridFilterManager.cs
@@ -118,7 +118,7 @@ public void UpdateFilterButtonStyles()
 
                     var textBlock = new TextBlock
                     {
-                        Text = "\uE71C",
+                        Text = hasActive ? "\uE16E" : "\uE71C",
                         FontFamily = new FontFamily("Segoe MDL2 Assets"),
                         Foreground = hasActive
                             ? new SolidColorBrush(Color.FromRgb(0xFF, 0xD7, 0x00))
diff --git a/Lite/Themes/DarkTheme.xaml b/Lite/Themes/DarkTheme.xaml
index b1bd6b07..f17ce306 100644
--- a/Lite/Themes/DarkTheme.xaml
+++ b/Lite/Themes/DarkTheme.xaml
@@ -357,9 +357,13 @@
     <!-- Column Filter Button Style when filter is active -->
     <Style x:Key="ColumnFilterButtonActiveStyle" TargetType="Button" BasedOn="{StaticResource ColumnFilterButtonStyle}">
         <Setter Property="Foreground" Value="#FFD700"/>
+        <Setter Property="Background" Value="#33FFD700"/>
+        <Setter Property="Content" Value="&#xE16E;"/>
+        <Setter Property="ToolTip" Value="Filter active — click to modify"/>
         <Style.Triggers>
             <Trigger Property="IsMouseOver" Value="True">
                 <Setter Property="Foreground" Value="#FFEC8B"/>
+                <Setter Property="Background" Value="#55FFD700"/>
             </Trigger>
         </Style.Triggers>
     </Style>