feat: UI polish, unified color scheme, remove debug spam

Author: adbergen

CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## [1.2.2] - 2025-02-03
+
+### Changed
+- **UI color scheme refresh**: Unified dark neutral theme replacing blue accents
+- **Tighter currency spacing**: Reduced spacing around separators in totals display
+
+### Fixed
+- **Debug messages removed**: Cleaned up chat spam from debug prints running outside debug mode
+
 ## [1.2.1] - 2025-02-03
 
 ### Added

Core.lua

@@ -111,7 +111,9 @@ function HonorLog:RestoreBGState()
         bgStartHonor = state.bgStartHonor
         bgHonorAccumulated = state.bgHonorAccumulated or 0
         isInBG = state.isInBG
-        print("|cff00ff00[HonorLog]|r Restored BG state: " .. tostring(currentBG) .. ", honor: " .. tostring(bgHonorAccumulated))
+        if debugMode then
+            print("|cff00ff00[HonorLog]|r Restored BG state: " .. tostring(currentBG) .. ", honor: " .. tostring(bgHonorAccumulated))
+        end
     end
 end
 
@@ -924,17 +926,11 @@ end
 
 -- Player logout - save current GetTime() for reload detection
 function HonorLog:PLAYER_LOGOUT()
-    -- Debug: always print when this fires
-    print("|cff00ff00[HonorLog]|r PLAYER_LOGOUT fired! GetTime()=" .. string.format("%.1f", GetTime()))
-
     if self.db and self.db.char then
         -- Store current GetTime() - on /reload this value will be less than GetTime() at next load
         -- On fresh login after logout, GetTime() will have reset, so it will be less than this stored value
         self.db.char.lastGameTime = GetTime()
         self.db.char.wasLogout = true -- Keep for backwards compatibility
-        print("|cff00ff00[HonorLog]|r   Saved lastGameTime=" .. string.format("%.1f", self.db.char.lastGameTime))
-    else
-        print("|cffff0000[HonorLog]|r   ERROR: self.db or self.db.char is nil!")
     end
 end
 

Data.lua

@@ -213,7 +213,7 @@ function HonorLog:InitializeDB()
     end
 
     if shouldResetSession then
-        print("|cff00ff00[HonorLog]|r New day detected - resetting daily stats for " .. date("%Y-%m-%d", now))
+        -- Silent reset - no chat spam
         HonorLogCharDB.session = DeepCopy(DEFAULTS.char.session)
         HonorLogCharDB.bgState = DeepCopy(DEFAULTS.char.bgState)
         HonorLogCharDB.sessionStartTime = 0 -- Deprecated, kept for backwards compatibility

HonorLog.toc

@@ -2,7 +2,7 @@
 ## Title: HonorLog
 ## Notes: Track battleground and world PvP statistics with gear goal tracking, hourly honor rates, and session/lifetime stats for TBC Classic Anniversary
 ## Author: adbergen
-## Version: 1.2.1
+## Version: 1.2.2
 ## SavedVariables: HonorLogDB
 ## SavedVariablesPerCharacter: HonorLogCharDB
 

UI/MainFrame.lua

@@ -70,15 +70,15 @@ local function CreateInnerShadow(frame)
     return shadow
 end
 
--- Create glow border
+-- Create glow border (rounded)
 local function CreateGlowBorder(frame, color)
     local glow = CreateFrame("Frame", nil, frame, "BackdropTemplate")
-    glow:SetPoint("TOPLEFT", -2, 2)
-    glow:SetPoint("BOTTOMRIGHT", 2, -2)
+    glow:SetPoint("TOPLEFT", -3, 3)
+    glow:SetPoint("BOTTOMRIGHT", 3, -3)
     glow:SetBackdrop({
         bgFile = nil,
-        edgeFile = "Interface\\ChatFrame\\ChatFrameBackground",
-        edgeSize = 2,
+        edgeFile = "Interface\\Tooltips\\UI-Tooltip-Border",
+        edgeSize = 12,
     })
     glow:SetBackdropBorderColor(unpack(color or COLORS.borderGlow))
     glow:SetFrameLevel(frame:GetFrameLevel() - 1)
@@ -185,11 +185,7 @@ local function CreateMainFrame()
         insets = { left = 4, right = 4, top = 4, bottom = 4 },
     })
     frame:SetBackdropColor(unpack(COLORS.bgPrimary))
-    frame:SetBackdropBorderColor(unpack(COLORS.borderDark))
-
-    -- Outer glow effect
-    local outerGlow = CreateGlowBorder(frame, COLORS.borderGlow)
-    frame.outerGlow = outerGlow
+    frame:SetBackdropBorderColor(0.18, 0.18, 0.22, 1) -- Clean dark border
 
     ----------------------------------------------------------------------------
     -- HEADER

UI/Theme.lua

@@ -18,13 +18,13 @@ HonorLog.Theme = {
     -- Accent gradients
     headerGradientTop = { 0.18, 0.18, 0.24, 1 },
     headerGradientBot = { 0.10, 0.10, 0.14, 1 },
-    accentGlow = { 0.30, 0.70, 0.95, 0.15 },
+    accentGlow = { 0.25, 0.25, 0.30, 0.15 },   -- Neutral (was blue)
 
     -- Borders
     borderDark = { 0.06, 0.06, 0.08, 1 },
     borderLight = { 0.30, 0.30, 0.38, 0.8 },
-    borderGlow = { 0.40, 0.75, 1.0, 0.25 },
-    borderAccent = { 0.35, 0.65, 0.90, 0.6 },
+    borderGlow = { 0.20, 0.22, 0.25, 0.4 },     -- Subtle dark glow (was blue)
+    borderAccent = { 0.30, 0.32, 0.35, 0.8 },   -- Neutral accent (was blue)
 
     -- Status colors
     win = { 0.30, 0.90, 0.40, 1 },
@@ -43,14 +43,14 @@ HonorLog.Theme = {
     textMuted = { 0.38, 0.38, 0.45, 1 },
 
     -- Brand accent
-    accent = { 0.35, 0.78, 1.0, 1 },
-    accentDim = { 0.25, 0.55, 0.75, 0.8 },
+    accent = { 0.85, 0.75, 0.45, 1 },          -- Warm gold for labels (was blue)
+    accentDim = { 0.25, 0.27, 0.30, 0.9 },     -- Dark neutral for borders (was blue)
     brand = { 0.25, 0.85, 0.45, 1 },
     brandDim = { 0.18, 0.60, 0.32, 0.8 },
 
     -- Separators
     separator = { 0.25, 0.25, 0.32, 0.6 },
-    separatorGlow = { 0.35, 0.65, 0.90, 0.2 },
+    separatorGlow = { 0.30, 0.30, 0.35, 0.3 },  -- Neutral (was blue)
 
     -- Progress colors (for goal tracking)
     progressFull = { 0.30, 0.90, 0.40, 1 },