fix wrapping of buff/debuff info text in compared Calcs tab

Author: oboking

src/Classes/CompareTab.lua

@@ -19,6 +19,33 @@ local configVisibility = LoadModule("Modules/ConfigVisibility")
 -- Node IDs below this value are normal passive tree nodes; IDs at or above are cluster jewel nodes
 local CLUSTER_NODE_OFFSET = 65536
 
+-- Wrap a string into lines for a given pixel width at font height 14 ("VAR").
+-- Breaks BEFORE a word that would exceed the width, so rendered lines never
+-- overshoot into the next column.
+local function wrapInfoLine(str, width)
+	local lines = {}
+	if not str or str == "" or width <= 0 then
+		return lines
+	end
+	local lineStart = 1
+	local pos = 1
+	while pos <= #str do
+		local ws, we = str:find("%s+", pos)
+		local wordEnd = ws and (ws - 1) or #str
+		if lineStart < pos and DrawStringWidth(14, "VAR", str:sub(lineStart, wordEnd)) > width then
+			local committed = (str:sub(lineStart, pos - 1):gsub("%s+$", ""))
+			t_insert(lines, committed)
+			lineStart = pos
+		end
+		if not ws then
+			t_insert(lines, str:sub(lineStart))
+			break
+		end
+		pos = we + 1
+	end
+	return lines
+end
+
 -- Layout constants (shared across Draw, DrawConfig, DrawItems, DrawCalcs, etc.)
 local LAYOUT = {
 	-- Main tab control bar
@@ -2144,12 +2171,21 @@ function CompareTabClass:LayoutCalcsSkillControls(vp, compareEntry)
 	local pOutput = primaryEnv and primaryEnv.player and primaryEnv.player.output
 	local cOutput = compareEnv and compareEnv.player and compareEnv.player.output
 	if pOutput or cOutput then
+		local wrapWidth = colWidth - 8
+		local infoLabels = {
+			BuffList = "Aura/Buff Skills",
+			CombatList = "Combat Buffs",
+			CurseList = "Curses/Debuffs",
+		}
 		local infoKeys = { "BuffList", "CombatList", "CurseList" }
 		for _, key in ipairs(infoKeys) do
 			local pVal = pOutput and pOutput[key]
 			local cVal = cOutput and cOutput[key]
 			if (pVal and pVal ~= "") or (cVal and cVal ~= "") then
-				textLinesHeight = textLinesHeight + 18
+				local label = infoLabels[key]
+				local pLines = (pVal and pVal ~= "") and #wrapInfoLine(label .. ": " .. pVal, wrapWidth) or 0
+				local cLines = (cVal and cVal ~= "") and #wrapInfoLine(label .. ": " .. cVal, wrapWidth) or 0
+				textLinesHeight = textLinesHeight + m_max(pLines, cLines, 1) * 18
 			end
 		end
 	end
@@ -4087,6 +4123,7 @@ function CompareTabClass:DrawCalcsSkillHeader(vp, compareEntry, headerHeight, pr
 	self.calcsSkillHeaderHover = nil  -- Reset hover state
 	if pOutput or cOutput then
 		local cursorX, cursorY = GetCursorPos()
+		local wrapWidth = colWidth - 8
 		local infoLines = {
 			{ label = "Aura/Buff Skills", key = "BuffList", breakdown = "SkillBuffs" },
 			{ label = "Combat Buffs", key = "CombatList" },
@@ -4096,33 +4133,42 @@ function CompareTabClass:DrawCalcsSkillHeader(vp, compareEntry, headerHeight, pr
 			local pVal = pOutput and pOutput[info.key]
 			local cVal = cOutput and cOutput[info.key]
 			if (pVal and pVal ~= "") or (cVal and cVal ~= "") then
+				local pLines = (pVal and pVal ~= "") and wrapInfoLine(info.label .. ": " .. pVal, wrapWidth) or {}
+				local cLines = (cVal and cVal ~= "") and wrapInfoLine(info.label .. ": " .. cVal, wrapWidth) or {}
+				local pH = #pLines * 18
+				local cH = #cLines * 18
+				local rowH = m_max(pH, cH, 18)
 				-- Check hover per-side for lines that have breakdown data
-				if info.breakdown and cursorY >= textY and cursorY < textY + 18 then
-					local onLeft = cursorX >= leftX and cursorX < rightX
-					local onRight = cursorX >= rightX and cursorX < vp.x + vp.width
+				if info.breakdown and cursorY >= textY and cursorY < textY + rowH then
+					local onLeft = cursorX >= leftX and cursorX < rightX and pH > 0 and cursorY < textY + pH
+					local onRight = cursorX >= rightX and cursorX < vp.x + vp.width and cH > 0 and cursorY < textY + cH
 					if onLeft then
 						SetDrawColor(0.15, 0.25, 0.15)
-						DrawImage(nil, leftX, textY, colWidth, 18)
+						DrawImage(nil, leftX, textY, colWidth, pH)
 						self.calcsSkillHeaderHover = {
 							breakdown = info.breakdown,
 							label = info.label,
 							build = self.primaryBuild,
-							x = leftX, y = textY, w = colWidth, h = 18,
+							x = leftX, y = textY, w = colWidth, h = pH,
 						}
 					elseif onRight then
 						SetDrawColor(0.15, 0.25, 0.15)
-						DrawImage(nil, rightX, textY, colWidth, 18)
+						DrawImage(nil, rightX, textY, colWidth, cH)
 						self.calcsSkillHeaderHover = {
 							breakdown = info.breakdown,
 							label = info.label,
 							build = compareEntry,
-							x = rightX, y = textY, w = colWidth, h = 18,
+							x = rightX, y = textY, w = colWidth, h = cH,
 						}
 					end
 				end
-				DrawString(leftX, textY + 1, "LEFT", 14, "VAR", "^7" .. info.label .. ": " .. (pVal or ""))
-				DrawString(rightX, textY + 1, "LEFT", 14, "VAR", "^7" .. info.label .. ": " .. (cVal or ""))
-				textY = textY + 18
+				for i, line in ipairs(pLines) do
+					DrawString(leftX, textY + 1 + (i - 1) * 18, "LEFT", 14, "VAR", "^7" .. line)
+				end
+				for i, line in ipairs(cLines) do
+					DrawString(rightX, textY + 1 + (i - 1) * 18, "LEFT", 14, "VAR", "^7" .. line)
+				end
+				textY = textY + rowH
 			end
 		end
 	end