Add files via upload

README.md

@@ -1,3 +1,6 @@
+<img src="https://cdn.discordapp.com/attachments/1510333697277300856/1525103487002869870/bg.png?ex=6a522ace&is=6a50d94e&hm=a0905d6bf7226f5871d6338582075e5d31e5396073909b064fb92d22ac217c90&"/>
+
+
 # Relationships PVP
 
 Relationships PVP is an all-in-one PvP companion addon for Vanilla OctoWoW.

RelationshipsPVP.lua

@@ -47,6 +47,17 @@ local ARENA_IDS = {
 local MARKS_WIN  = 3
 local MARKS_LOSS = 1
 
+-- TurtleWoW / OctoWoW (Patch 1.18.1+) use an accumulative honor system.
+-- The value reported on the battleground scoreboard (honorGained from
+-- GetBattlefieldScore) is the old-style number, but only 10% of it is
+-- actually credited to the player as spendable Honor currency. Costs of
+-- gear/items were reduced to match. So spendable = floor(scoreboard * 0.10).
+local HONOR_SPEND_FACTOR = 0.10
+
+local function SpendableHonor(h)
+  return math.floor((h or 0) * HONOR_SPEND_FACTOR)
+end
+
 local currentBG = nil
 local endReported = {}
 
@@ -63,7 +74,7 @@ local totalRow   = nil
 ---------------------------------------------------------------
 local function NewStatBlock()
   return {
-    wins = 0, losses = 0,
+    wins = 0, draws = 0, losses = 0,
     kills = 0, deaths = 0, hk = 0,
     marks = 0, honor = 0,
     bestKB = 0, streak = 0, bestStreak = 0,
@@ -84,20 +95,91 @@ local function EnsureDB()
     if s.bestStreak == nil then s.bestStreak = 0 end
     if s.hk         == nil then s.hk         = 0 end
     if s.marks      == nil then s.marks      = 0 end
+    if s.draws      == nil then s.draws      = 0 end
     db.stats[k] = s
   end
   return db
 end
 
+-- Clamp a UIParent-relative CENTER offset so the button stays visible on screen.
+function RelationshipsPVP_ClampScreenOffset(x, y)
+  x = x or 0
+  y = y or 0
+  if not UIParent or not UIParent.GetWidth then return x, y end
+  local w = UIParent:GetWidth()  or 1024
+  local h = UIParent:GetHeight() or 768
+  local pad = -60
+  local maxX = (w / 2) - pad
+  local maxY = (h / 2) - pad
+  if x >  maxX then x =  maxX end
+  if x < -maxX then x = -maxX end
+  if y >  maxY then y =  maxY end
+  if y < -maxY then y = -maxY end
+  return x, y
+end
+
+-- Position the icon anywhere on screen (parented to UIParent for free drag).
+function RelationshipsPVP_SetIconPosition(x, y, save)
+  local btn = RelationshipsPVPMinimapButton
+  if not btn or not UIParent then return end
+  x, y = RelationshipsPVP_ClampScreenOffset(x, y)
+  btn:ClearAllPoints()
+  btn:SetParent(UIParent)
+  btn:SetPoint("CENTER", UIParent, "CENTER", x, y)
+  if btn.SetFrameStrata then btn:SetFrameStrata("MEDIUM") end
+  if btn.SetFrameLevel then btn:SetFrameLevel(10) end
+  if save then
+    local db = EnsureDB()
+    db.icon.point    = "CENTER"
+    db.icon.relPoint = "CENTER"
+    db.icon.parent   = "UIParent"
+    db.icon.x        = x
+    db.icon.y        = y
+  end
+end
+
+-- Read the button's current center in UIParent coordinates and persist it.
 function RelationshipsPVP_SaveIconPos()
   local db = EnsureDB()
+  local btn = RelationshipsPVPMinimapButton
+  if not btn or not UIParent then return end
+  local x = db.icon.x or 0
+  local y = db.icon.y or 0
+  if btn.GetCenter and UIParent.GetCenter then
+    local bx, by = btn:GetCenter()
+    local ux, uy = UIParent:GetCenter()
+    if bx and by and ux and uy then
+      x = bx - ux
+      y = by - uy
+    end
+  end
+  x, y = RelationshipsPVP_ClampScreenOffset(x, y)
+  db.icon.point    = "CENTER"
+  db.icon.relPoint = "CENTER"
+  db.icon.parent   = "UIParent"
+  db.icon.x        = x
+  db.icon.y        = y
+end
+
+-- Free drag: reparent to UIParent and let the client move the frame.
+function RelationshipsPVP_StartIconDrag()
   local btn = RelationshipsPVPMinimapButton
   if not btn then return end
-  local point, _, _, x, y = btn:GetPoint()
-  db.icon.point  = point or "CENTER"
-  db.icon.x      = x or 0
-  db.icon.y      = y or 0
-  db.icon.parent = "Minimap"
+  btn:SetParent(UIParent)
+  if btn.SetFrameStrata then btn:SetFrameStrata("MEDIUM") end
+  if btn.SetFrameLevel then btn:SetFrameLevel(10) end
+  if btn.StartMoving then btn:StartMoving() end
+end
+
+function RelationshipsPVP_StopIconDrag()
+  local btn = RelationshipsPVPMinimapButton
+  if not btn then return end
+  if btn.StopMovingOrSizing then btn:StopMovingOrSizing() end
+  RelationshipsPVP_SaveIconPos()
+  local db = EnsureDB()
+  -- Re-anchor cleanly to a single CENTER/CENTER offset so the saved value is
+  -- exactly what we restore on next login.
+  RelationshipsPVP_SetIconPosition(db.icon.x, db.icon.y, 1)
 end
 
 local function ApplyIconVisibility()
@@ -107,10 +189,8 @@ local function ApplyIconVisibility()
   if db.showIcon then btn:Show() else btn:Hide() end
 end
 
--- The Minimap frame can be briefly hidden during loading screens / entering
--- battlegrounds; when it re-shows, child buttons parented to it sometimes
--- stay hidden until touched. This ticker re-asserts our intended visibility
--- once per second so the icon never "randomly disappears".
+-- Loading screens / entering BGs can briefly hide parent frames; re-assert
+-- our intended visibility so the icon never "randomly disappears".
 local iconWatcher
 local function InstallIconWatcher()
   if iconWatcher then return end
@@ -136,9 +216,22 @@ local function RestoreIconPos()
   local db = EnsureDB()
   local btn = RelationshipsPVPMinimapButton
   if not btn then return end
-  local parent = (db.icon.parent == "UIParent") and UIParent or Minimap
-  btn:ClearAllPoints()
-  btn:SetPoint(db.icon.point, parent, db.icon.point, db.icon.x, db.icon.y)
+
+  -- Migrate any prior Minimap-relative save into a UIParent-relative offset
+  -- so free-drag mode has a valid starting position.
+  if db.icon.parent == "Minimap" and Minimap and Minimap.GetCenter and UIParent and UIParent.GetCenter then
+    local mx, my = Minimap:GetCenter()
+    local ux, uy = UIParent:GetCenter()
+    if mx and my and ux and uy then
+      db.icon.x = (mx - ux) + (db.icon.x or 0)
+      db.icon.y = (my - uy) + (db.icon.y or 0)
+    end
+    db.icon.parent   = "UIParent"
+    db.icon.point    = "CENTER"
+    db.icon.relPoint = "CENTER"
+  end
+
+  RelationshipsPVP_SetIconPosition(db.icon.x or 0, db.icon.y or 0, 1)
   ApplyIconVisibility()
   InstallIconWatcher()
 end
@@ -228,22 +321,33 @@ local function RecordEnd(key, index)
   if not s then return end
 
   local playerWon = false
+  local isDraw = false
   local winner = GetBattlefieldWinner()
   if winner then
     local myFaction = UnitFactionGroup("player")
-    local wonAsAlliance = (winner == 1 and myFaction == "Alliance")
-    local wonAsHorde    = (winner == 0 and myFaction == "Horde")
-    playerWon = wonAsAlliance or wonAsHorde
-    if playerWon then
-      s.wins   = s.wins + 1
-      s.streak = s.streak + 1
-      if s.streak > s.bestStreak then s.bestStreak = s.streak end
-    else
-      s.losses = s.losses + 1
+    -- Some server builds (incl. Turtle/Octo) report a draw as a winner
+    -- value that is neither 0 (Horde) nor 1 (Alliance). Treat anything
+    -- else as a draw so it isn't miscounted as a loss.
+    isDraw = (winner ~= 0 and winner ~= 1)
+    if isDraw then
+      s.draws  = s.draws + 1
       s.streak = 0
+    else
+      local wonAsAlliance = (winner == 1 and myFaction == "Alliance")
+      local wonAsHorde    = (winner == 0 and myFaction == "Horde")
+      playerWon = wonAsAlliance or wonAsHorde
+      if playerWon then
+        s.wins   = s.wins + 1
+        s.streak = s.streak + 1
+        if s.streak > s.bestStreak then s.bestStreak = s.streak end
+      else
+        s.losses = s.losses + 1
+        s.streak = 0
+      end
     end
     local isBG = (key == "AV" or key == "WSG" or key == "AB")
     if isBG then
+      -- Draws award the loss-tier mark count.
       s.marks = s.marks + (playerWon and MARKS_WIN or MARKS_LOSS)
     end
   end
@@ -271,9 +375,15 @@ local function RecordEnd(key, index)
   if key == "AV" or key == "WSG" or key == "AB" then
     marksMsg = " +" .. (playerWon and MARKS_WIN or MARKS_LOSS) .. " marks"
   end
-  Print(BG_NAMES[key] .. " recorded ("
-    .. (playerWon and "|cff66ff66win|r" or "|cffff6666loss|r")
-    .. marksMsg .. ").")
+  local outcomeStr
+  if isDraw then
+    outcomeStr = "|cffffff66draw|r"
+  elseif playerWon then
+    outcomeStr = "|cff66ff66win|r"
+  else
+    outcomeStr = "|cffff6666loss|r"
+  end
+  Print(BG_NAMES[key] .. " recorded (" .. outcomeStr .. marksMsg .. ").")
 end
 
 ---------------------------------------------------------------
@@ -281,41 +391,71 @@ end
 ---------------------------------------------------------------
 -- Auto-close the Blizzard PvP panels if they pop up as a side effect
 -- of the arena / BG queueing calls.
+-- NOTE: We deliberately do NOT call HideUIPanel() here. HideUIPanel on a
+-- frame that Blizzard's UI panel system has slotted into a left/center
+-- slot triggers a slot reflow, which is what caused Character / Friends /
+-- Spellbook panels to jump to the middle of the screen after the PvP
+-- panel was opened. Directly calling :Hide() bypasses the slot manager
+-- entirely; the slot suppression below (SuppressUIPanelWindows) ensures
+-- Blizzard never registered a slot for these frames to begin with.
 local function HidePvPPanels()
-  if BattlefieldFrame       and BattlefieldFrame:IsVisible()       then HideUIPanel(BattlefieldFrame)       end
-  if PVPBattlegroundFrame   and PVPBattlegroundFrame:IsVisible()   then HideUIPanel(PVPBattlegroundFrame)   end
-  if PVPFrame               and PVPFrame:IsVisible()               then HideUIPanel(PVPFrame)               end
-  if HonorFrame             and HonorFrame:IsVisible()             then HideUIPanel(HonorFrame)             end
+  if BattlefieldFrame       and BattlefieldFrame:IsVisible()       then BattlefieldFrame:Hide()       end
+  if PVPBattlegroundFrame   and PVPBattlegroundFrame:IsVisible()   then PVPBattlegroundFrame:Hide()   end
+  if PVPFrame               and PVPFrame:IsVisible()               then PVPFrame:Hide()               end
+  if HonorFrame             and HonorFrame:IsVisible()             then HonorFrame:Hide()             end
 end
 
 -- Suppress the PvP panel entirely while we are auto-queueing. Reference-counted
 -- so nested queues (Queue All BGs) do not release too early.
+--
+-- IMPORTANT: We previously hijacked BattlefieldFrame.Show / PVPBattlegroundFrame.Show
+-- to no-op during suppression. That was the source of the "Character / Friends
+-- panels jump to the middle of the screen" bug: Blizzard's ShowUIPanel() first
+-- allocates a UI panel slot (left / center / doublewide) via UIPanelWindows,
+-- THEN calls frame:Show(). Silently dropping :Show() left the slot reserved by
+-- an invisible frame, so the next panel the user opened (Character, Friends,
+-- Spellbook, ...) got shoved into a different slot -> centered.
+--
+-- The correct approach on 1.12 is to temporarily remove these frames from
+-- UIPanelWindows while suppressing, so ShowUIPanel treats them as plain
+-- frames and never allocates a slot. When suppression ends, the original
+-- UIPanelWindows entries are restored so normal Blizzard UI behaviour is
+-- preserved for the rest of the session.
 local suppressPanels = 0
-local suppressHooked = false
-local origBattlefieldShow, origPVPBGShow
+local savedUIPanelEntries = nil
 
-local function InstallPanelSuppressor()
-  if suppressHooked then return end
-  suppressHooked = true
-  if BattlefieldFrame then
-    origBattlefieldShow = BattlefieldFrame.Show
-    BattlefieldFrame.Show = function(self, ...)
-      if suppressPanels > 0 then return end
-      origBattlefieldShow(self)
-    end
-  end
-  if PVPBattlegroundFrame then
-    origPVPBGShow = PVPBattlegroundFrame.Show
-    PVPBattlegroundFrame.Show = function(self, ...)
-      if suppressPanels > 0 then return end
-      origPVPBGShow(self)
-    end
+local SUPPRESSED_PANEL_NAMES = {
+  "BattlefieldFrame",
+  "PVPBattlegroundFrame",
+  "PVPFrame",
+  "HonorFrame",
+}
+
+local function SuppressUIPanelWindows()
+  if savedUIPanelEntries then return end
+  savedUIPanelEntries = {}
+  if type(UIPanelWindows) ~= "table" then return end
+  for i = 1, table.getn(SUPPRESSED_PANEL_NAMES) do
+    local name = SUPPRESSED_PANEL_NAMES[i]
+    savedUIPanelEntries[name] = UIPanelWindows[name]
+    UIPanelWindows[name] = nil
   end
 end
 
+local function RestoreUIPanelWindows()
+  if not savedUIPanelEntries then return end
+  if type(UIPanelWindows) == "table" then
+    for i = 1, table.getn(SUPPRESSED_PANEL_NAMES) do
+      local name = SUPPRESSED_PANEL_NAMES[i]
+      UIPanelWindows[name] = savedUIPanelEntries[name]
+    end
+  end
+  savedUIPanelEntries = nil
+end
+
 local function BeginSuppress()
-  InstallPanelSuppressor()
   suppressPanels = suppressPanels + 1
+  SuppressUIPanelWindows()
   HidePvPPanels()
 end
 
@@ -323,6 +463,9 @@ local function EndSuppress()
   suppressPanels = suppressPanels - 1
   if suppressPanels < 0 then suppressPanels = 0 end
   HidePvPPanels()
+  if suppressPanels == 0 then
+    RestoreUIPanelWindows()
+  end
 end
 
 -- Central pending-queue state. `pendingQueue` is a FIFO of items:
@@ -682,10 +825,10 @@ end
 ---------------------------------------------------------------
 -- UI: build row FontStrings inside the stats panel
 ---------------------------------------------------------------
-local COL_X = {  14, 158, 208, 258, 308, 354, 400, 446, 496, 550, 614 }
-local COL_W = { 140,  46,  46,  46,  46,  46,  46,  46,  50,  60,  58 }
-local COL_J = { "LEFT", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER" }
-local COL_NAMES = { "bg", "games", "win", "loss", "ratio", "kb", "hk", "dth", "kd", "honor", "mk" }
+local COL_X = {  14, 158, 208, 258, 308, 358, 408, 454, 500, 550, 604, 664, 744 }
+local COL_W = { 140,  46,  46,  46,  46,  46,  46,  46,  46,  50,  56,  76,  56 }
+local COL_J = { "LEFT", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER", "CENTER" }
+local COL_NAMES = { "bg", "games", "win", "draw", "loss", "ratio", "kb", "hk", "dth", "kd", "honor", "spend", "mk" }
 
 local ROW_HEIGHT = 22
 local ROW_TOP    = -34
@@ -695,7 +838,7 @@ local function MakeRow(panel, tag, yOff, striped)
   if striped then
     local bg = panel:CreateTexture("RPVPRowBG"..tag, "BACKGROUND")
     bg:SetTexture(1, 1, 1, 0.04)
-    bg:SetWidth(660); bg:SetHeight(ROW_HEIGHT - 2)
+    bg:SetWidth(786); bg:SetHeight(ROW_HEIGHT - 2)
     bg:SetPoint("TOPLEFT", panel, "TOPLEFT", 14, yOff + 2)
   end
   for c = 1, table.getn(COL_NAMES) do
@@ -731,7 +874,7 @@ local function BuildStatsRows()
   local totalY = ROW_TOP - n * ROW_HEIGHT - 4
   local div = panel:CreateTexture("RPVPTotalDivider", "ARTWORK")
   div:SetTexture(1, 1, 1, 0.25)
-  div:SetWidth(660); div:SetHeight(1)
+  div:SetWidth(786); div:SetHeight(1)
   div:SetPoint("TOPLEFT", panel, "TOPLEFT", 14, totalY + 2)
 
   totalRow = MakeRow(panel, "TOTAL", totalY - 6, false)
@@ -766,7 +909,8 @@ local function FormatQueueLine()
 end
 
 local function FillRow(r, s)
-  local games = s.wins + s.losses
+  local draws = s.draws or 0
+  local games = s.wins + draws + s.losses
   local pct = 0
   if games > 0 then pct = math.floor((s.wins / games) * 100 + 0.5) end
   local kd = "-"
@@ -778,6 +922,7 @@ local function FillRow(r, s)
 
   r.games:SetText(games)
   r.win:SetText(s.wins)
+  r.draw:SetText(draws)
   r.loss:SetText(s.losses)
   r.ratio:SetText(pct .. "%")
   r.kb:SetText(s.kills)
@@ -785,6 +930,7 @@ local function FillRow(r, s)
   r.dth:SetText(s.deaths)
   r.kd:SetText(kd)
   r.honor:SetText(FormatNumber(s.honor))
+  r.spend:SetText(FormatNumber(SpendableHonor(s.honor)))
   r.mk:SetText(s.marks)
 
   if games == 0 then
@@ -811,6 +957,7 @@ function RelationshipsPVP_Refresh()
     if r then FillRow(r, s) end
 
     totals.wins   = totals.wins   + s.wins
+    totals.draws  = totals.draws  + (s.draws or 0)
     totals.losses = totals.losses + s.losses
     totals.kills  = totals.kills  + s.kills
     totals.hk     = totals.hk     + s.hk
@@ -828,6 +975,7 @@ function RelationshipsPVP_Refresh()
       "|cffffd200Best win streak:|r " .. totals.bestStreak
       .. "    |cffffd200Best KBs (single match):|r " .. totals.bestKB
       .. "    |cffffd200Total honor:|r " .. FormatNumber(totals.honor)
+      .. "    |cffffd200Spendable honor:|r " .. FormatNumber(SpendableHonor(totals.honor))
     )
   end
 
@@ -853,12 +1001,12 @@ function RelationshipsPVP_ResetPositions()
   end
 
   if RelationshipsPVPMinimapButton then
-    RelationshipsPVPMinimapButton:ClearAllPoints()
-    RelationshipsPVPMinimapButton:SetPoint("CENTER", UIParent, "CENTER", 0, 0)
-    db.icon.point  = "CENTER"
-    db.icon.x      = 0
-    db.icon.y      = 0
-    db.icon.parent = "UIParent"
+    RelationshipsPVP_SetIconPosition(0, 0, 1)
+    db.icon.point    = "CENTER"
+    db.icon.relPoint = "CENTER"
+    db.icon.x        = 0
+    db.icon.y        = 0
+    db.icon.parent   = "UIParent"
   end
 
   db.panel = { point = "CENTER", x = 0, y = 0 }
@@ -880,6 +1028,7 @@ end
 function RelationshipsPVP_OnLoad()
   this:RegisterEvent("PLAYER_LOGIN")
   this:RegisterEvent("PLAYER_ENTERING_WORLD")
+  this:RegisterEvent("PLAYER_LOGOUT")
   this:RegisterEvent("UPDATE_BATTLEFIELD_STATUS")
   this:RegisterEvent("UPDATE_BATTLEFIELD_SCORE")
   this:RegisterEvent("PLAYER_DEAD")
@@ -934,6 +1083,12 @@ function RelationshipsPVP_SavePanelPos()
 end
 
 function RelationshipsPVP_OnEvent(event)
+  if event == "PLAYER_LOGOUT" then
+    RelationshipsPVP_SaveIconPos()
+    SavePanelPos()
+    return
+  end
+
   if event == "PLAYER_LOGIN" or event == "PLAYER_ENTERING_WORLD" then
     EnsureDB()
     RestoreIconPos()

RelationshipsPVP.toc

@@ -2,7 +2,7 @@
 ## Title: Relationships PVP
 ## Notes: PvP panel with per-battleground and per-arena stats, marks, kills/deaths, honor, K/D, and queue controls. Supports the Battleground Finder and Arena on OctoWoW.
 ## Author: Relationship
-## Version: 1.0
+## Version: 1.3
 ## SavedVariablesPerCharacter: RelationshipsPVPCharDB
 
 RelationshipsPVP.xml

RelationshipsPVP.xml

@@ -6,7 +6,7 @@
        ============================================================ -->
   <Frame name="RelationshipsPVPFrame" parent="UIParent" toplevel="true" movable="true" enableMouse="true" hidden="true" frameStrata="HIGH">
     <Size>
-      <AbsDimension x="740" y="740"/>
+      <AbsDimension x="864" y="740"/>
     </Size>
     <Anchors>
       <Anchor point="CENTER"/>
@@ -74,7 +74,7 @@
 
       <!-- ================= Stats Panel ================= -->
       <Frame name="RelationshipsPVPStatsPanel">
-        <Size><AbsDimension x="692" y="266"/></Size>
+        <Size><AbsDimension x="816" y="266"/></Size>
         <Anchors>
           <Anchor point="TOPLEFT">
             <Offset><AbsDimension x="24" y="-72"/></Offset>
@@ -99,47 +99,55 @@
               <Size><AbsDimension x="46" y="14"/></Size>
               <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="208" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
-            <FontString name="RPVPHdrLoss"  inherits="GameFontNormalSmall" text="Loss"  justifyH="CENTER">
+            <FontString name="RPVPHdrDraw"  inherits="GameFontNormalSmall" text="Draw"  justifyH="CENTER">
               <Size><AbsDimension x="46" y="14"/></Size>
               <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="258" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
-            <FontString name="RPVPHdrRatio" inherits="GameFontNormalSmall" text="Win%" justifyH="CENTER">
+            <FontString name="RPVPHdrLoss"  inherits="GameFontNormalSmall" text="Loss"  justifyH="CENTER">
               <Size><AbsDimension x="46" y="14"/></Size>
               <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="308" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
+            <FontString name="RPVPHdrRatio" inherits="GameFontNormalSmall" text="Win%" justifyH="CENTER">
+              <Size><AbsDimension x="46" y="14"/></Size>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="358" y="-10"/></Offset></Anchor></Anchors>
+            </FontString>
             <FontString name="RPVPHdrKB"    inherits="GameFontNormalSmall" text="KB"    justifyH="CENTER">
               <Size><AbsDimension x="46" y="14"/></Size>
-              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="354" y="-10"/></Offset></Anchor></Anchors>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="408" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
             <FontString name="RPVPHdrHK"    inherits="GameFontNormalSmall" text="HK"    justifyH="CENTER">
               <Size><AbsDimension x="46" y="14"/></Size>
-              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="400" y="-10"/></Offset></Anchor></Anchors>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="454" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
             <FontString name="RPVPHdrDth"   inherits="GameFontNormalSmall" text="Deaths" justifyH="CENTER">
               <Size><AbsDimension x="46" y="14"/></Size>
-              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="446" y="-10"/></Offset></Anchor></Anchors>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="500" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
             <FontString name="RPVPHdrKD"    inherits="GameFontNormalSmall" text="K/D" justifyH="CENTER">
               <Size><AbsDimension x="50" y="14"/></Size>
-              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="496" y="-10"/></Offset></Anchor></Anchors>
-            </FontString>
-            <FontString name="RPVPHdrHonor" inherits="GameFontNormalSmall" text="Honor" justifyH="CENTER">
-              <Size><AbsDimension x="60" y="14"/></Size>
               <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="550" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
+            <FontString name="RPVPHdrHonor" inherits="GameFontNormalSmall" text="Honor" justifyH="CENTER">
+              <Size><AbsDimension x="56" y="14"/></Size>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="604" y="-10"/></Offset></Anchor></Anchors>
+            </FontString>
+            <FontString name="RPVPHdrSpend" inherits="GameFontNormalSmall" text="Spendable" justifyH="CENTER">
+              <Size><AbsDimension x="76" y="14"/></Size>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="664" y="-10"/></Offset></Anchor></Anchors>
+            </FontString>
             <FontString name="RPVPHdrMk"    inherits="GameFontNormalSmall" text="Marks" justifyH="CENTER">
-              <Size><AbsDimension x="58" y="14"/></Size>
-              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="614" y="-10"/></Offset></Anchor></Anchors>
+              <Size><AbsDimension x="56" y="14"/></Size>
+              <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="744" y="-10"/></Offset></Anchor></Anchors>
             </FontString>
 
             <Texture name="RPVPHdrDivider" file="Interface\Tooltips\UI-Tooltip-Background">
-              <Size><AbsDimension x="664" y="1"/></Size>
+              <Size><AbsDimension x="786" y="1"/></Size>
               <Anchors><Anchor point="TOPLEFT"><Offset><AbsDimension x="14" y="-28"/></Offset></Anchor></Anchors>
               <Color r="0.6" g="0.6" b="0.6" a="0.6"/>
             </Texture>
 
             <FontString name="RelationshipsPVPExtrasText" inherits="GameFontHighlightSmall" justifyH="LEFT">
-              <Size><AbsDimension x="664" y="16"/></Size>
+              <Size><AbsDimension x="786" y="16"/></Size>
               <Anchors>
                 <Anchor point="BOTTOMLEFT"><Offset><AbsDimension x="14" y="10"/></Offset></Anchor>
               </Anchors>
@@ -150,7 +158,7 @@
 
       <!-- ================= Queue Status Panel ================= -->
       <Frame name="RelationshipsPVPQueuePanel">
-        <Size><AbsDimension x="692" y="84"/></Size>
+        <Size><AbsDimension x="816" y="84"/></Size>
         <Anchors>
           <Anchor point="TOPLEFT">
             <Offset><AbsDimension x="24" y="-368"/></Offset>
@@ -164,7 +172,7 @@
         <Layers>
           <Layer level="ARTWORK">
             <FontString name="RelationshipsPVPQueueText" inherits="GameFontHighlightSmall" justifyH="LEFT" justifyV="TOP">
-              <Size><AbsDimension x="664" y="70"/></Size>
+              <Size><AbsDimension x="786" y="70"/></Size>
               <Anchors>
                 <Anchor point="TOPLEFT"><Offset><AbsDimension x="14" y="-10"/></Offset></Anchor>
               </Anchors>
@@ -173,15 +181,9 @@
         </Layers>
       </Frame>
 
-      <!-- ================= Action Panel =================
-           Layout math (panel width 692, side padding 24 → usable 644):
-             Row 1 (3 BGs)     : 3 × 200w + 2 × 22gap = 644
-             Row 2 (4 arena)   : 4 × 149w + 3 × 16gap = 596 + 48 = 644
-             Row 4 (4 utility) : 4 × 149w + 3 × 16gap = 644
-             All rows share x=24 left edge and end at x=668 right edge.
-      -->
+      <!-- ================= Action Panel ================= -->
       <Frame name="RelationshipsPVPActionPanel">
-        <Size><AbsDimension x="692" y="230"/></Size>
+        <Size><AbsDimension x="816" y="230"/></Size>
         <Anchors>
           <Anchor point="TOPLEFT">
             <Offset><AbsDimension x="24" y="-472"/></Offset>
@@ -280,7 +282,7 @@
             </Scripts>
           </CheckButton>
 
-          <!-- Row 4: Utility buttons (4 equal-width columns matching arena row) -->
+          <!-- Row 4: Utility buttons -->
           <Button name="RelationshipsPVPLeave" inherits="UIPanelButtonTemplate" text="Leave / Cancel Queue">
             <Size><AbsDimension x="149" y="30"/></Size>
             <Anchors>
@@ -383,11 +385,8 @@
         if icon and icon.SetTexCoord then
           icon:SetTexCoord(0.08, 0.92, 0.08, 0.92)
         end
-        this:SetScript("OnDragStart", function() this:StartMoving() end)
-        this:SetScript("OnDragStop", function()
-          this:StopMovingOrSizing()
-          RelationshipsPVP_SaveIconPos()
-        end)
+        this:SetScript("OnDragStart", function() RelationshipsPVP_StartIconDrag() end)
+        this:SetScript("OnDragStop", function() RelationshipsPVP_StopIconDrag() end)
       </OnLoad>
       <OnClick>
         if RelationshipsPVPFrame:IsVisible() then