wow/ouroloot: gui.lua comparison

comparison gui.lua @ 25:cb9635999171

- Reassigning loot, and marking loot as disenchanted/vault, will update the history entries also. - Tooltip help on dropdown menus appears at mouse now, instead of depending on Blizzard's "Beginner Tooltips" setting. - Forum BBcode output includes an option for MMO-Champion/Wowstead formatting. - Smarter cleanup functions for expiring caches. - Properly prefer locally-generated loot events, even when other users can see and rebroadcast the events back to you faster than you see them.

author	Farmbuyer of US-Kilrogg <farmbuyer@gmail.com>
date	Wed, 05 Oct 2011 02:14:07 +0000
parents	61d932f0e8f2
children	f866daadcdf6

comparison

equal deleted inserted replaced

-:61d932f0e8f2
+:cb9635999171
 				text = name,
 				func = dropdownmenu_handler,
 				arg1 = funcs[name],
 				arg2 = arg,
 				notCheckable = true,
+				tooltipOnButton = true,
+				tooltipWhileDisabled = true,
 				tooltipTitle = tiptext and name or nil,
 				tooltipText = tiptext,
 			})
 		end
 	end
 	["Rebroadcast this boss"] = function(rowi,kind)
 		addon:Print("not implemented yet") -- TODO
 	end,
 	["Mark as normal"] = function(rowi,disp) -- broadcast the change?  ugh
+		local olddisp = g_loot[rowi].disposition
 		g_loot[rowi].disposition = disp
 		g_loot[rowi].bcast_from = nil
 		g_loot[rowi].extratext = nil
+		addon:history_handle_disposition (rowi, olddisp)
 	end,
 	["Show only this player"] = function(rowi)
 		local st = _d:GetUserData("eoiST")
 		_d:SetUserData("player filter name", g_loot[rowi].person)
 	df_REASSIGN = function(rowi,to_whom)
 		addon:reassign_loot (rowi, to_whom)
 		CloseDropDownMenus()  -- also need to close parent menu
 	end,
 	["Enter name..."] = function(rowi)
+		CloseDropDownMenus()  -- also need to close parent menu
 		local dialog = StaticPopup_Show "OUROL_REASSIGN_ENTER"
 		dialog.data = {index=rowi, display=_d}
 	end,
 }
 -- Would be better to move the %arg to this list rather than below, but
 		isTitle = true,
 		notClickable = true,
 		notCheckable = true,
 	}},
 	{
-		"Rebroadcast this day%time|Broadcasts everything from here down until a new day",
+		"Rebroadcast this day%time|Broadcasts everything from here down until a new day.",
-		"Delete remaining entries for this day%time|Erases everything from here down until a new day",
+		"Delete remaining entries for this day%time|Erases everything from here down until a new day.",
-		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information",
+		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information.",
-		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information",
+		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information.",
 		CLOSE
 	}, dropdownfuncs)
 local eoi_loot_dropdown = gen_easymenu_table(
 	{{
 		-- this is the dropdown title, text filled in on the fly
 	}},
 	{
 		"Mark as disenchanted%shard",
 		"Mark as offspec%offspec",
 		"Mark as guild vault%gvault",
-		"Mark as normal|This is the default. Selecting any 'Mark as <x>' action blanks out extra notes about who broadcast this entry, etc.",
+		"Mark as normal|This is the default.  Selecting any 'Mark as <x>' action blanks out extra notes about who broadcast this entry, etc.",
 		"--",
 		"Rebroadcast this loot entry|Sends this loot event, including special notes, as if it just happened.",
 		"Delete this loot event|Permanent, no going back!",
-		"Delete remaining entries for this boss%boss|Erases everything from here down until a new boss/day",
+		"Delete remaining entries for this boss%boss|Erases everything from here down until a new boss/day.",
-		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information",
+		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information.",
-		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information",
+		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information.",
-		"Edit note|Same as double-clicking in the notes column",
+		"Edit note|Same as double-clicking in the notes column.",
 		"--",
 		CLOSE
 	}, dropdownfuncs)
 local eoi_player_dropdown = gen_easymenu_table(
 	{
 		},
 		{
 			text = "Reassign to...",
 			hasArrow = true,
 			--menuList = filled in in the fly,
+			tooltipOnButton = true,
+			tooltipWhileDisabled = true,
 		},
 	},
 	{
 		"Show only this player",
 		CLOSE
 		isTitle = true,
 		notClickable = true,
 		notCheckable = true,
 	}},
 	{
-		"Change from 'wipe' to 'kill'|Also collapses other wipe entries",
+		"Change from 'wipe' to 'kill'|Also collapses other wipe entries.",
-		"Rebroadcast this boss|Broadcasts the kill event and all subsequent loot until next boss",
+		"Rebroadcast this boss|Broadcasts the kill event and all subsequent loot until next boss.",
 		"Delete this boss event|Permanent, no going back!",
-		"Delete remaining entries for this boss%boss|Erases everything from here down until a new boss/day",
+		"Delete remaining entries for this boss%boss|Erases everything from here down until a new boss/day.",
-		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information",
+		"Insert new loot entry%loot|Inserts new loot above this one, prompting you for information.",
-		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information",
+		"Insert new boss kill event%boss|Inserts new event above this one, prompting you for information.",
 		"--",
 		CLOSE
 	}, dropdownfuncs)
 local hist_dropdown = gen_easymenu_table(
 	{{
 				notCheckable = true,
 			}
 		end
 		eoi_player_dropdown[2].menuList =
 			gen_easymenu_table (raiders, {"Enter name...",CLOSE}, dropdownfuncs)
-		--tabledump(eoi_player_dropdown)
+		if e.disposition == 'shard' or e.disposition == 'gvault' then
+			eoi_player_dropdown[2].disabled = true
+			eoi_player_dropdown[2].tooltipTitle = "Cannot Reassign"
+			eoi_player_dropdown[2].tooltipText = "You must first mark this item as 'normal' or 'offspec' before reassignment."
+		else
+			eoi_player_dropdown[2].disabled = nil
+			eoi_player_dropdown[2].tooltipTitle = nil
+			eoi_player_dropdown[2].tooltipText = nil
+		end
 		EasyMenu (eoi_player_dropdown, dropdownmenuframe, cellFrame, 0, 0, "MENU")
 	elseif kind == 'boss' then
 		eoi_boss_dropdown[1].text = e.bosskill
 		EasyMenu (eoi_boss_dropdown, dropdownmenuframe, cellFrame, 0, 0, "MENU")
 		return true  -- do not do anything further
 	end]]
 	tabs_OnGroupSelected["hist"] = function(container,specials)
 		histST = LibStub("ScrollingTable"):CreateST(hist_st_cols,eoi_st_displayed_rows,eoi_st_rowheight)
+		_d:SetUserData("histST",histST)
 		if addon.author_debug then
 			_G.OLHST = histST
 		end
 		if not eoi_st_otherrow_bgcolortable_default then
 			histST:SetData(addon.history.st)
 		end
 		tabs_OnGroupSelected["hist"] = function(container,specials)
 			local st_widget = GUI:Create("lib-st")
+			-- don't need _d:GetUserData("histST") here, as it's already a local
 			histST:OuroLoot_Refresh()
 			st_widget:WrapST(histST)
 			st_widget.head_offset = 15
 			st_widget.tail_offset = 0
 			container:SetLayout("Fill")
 	if self.display then
 		-- try to get everything to update, rebuild, refresh... ugh, no
 		self.display:Hide()
 	end
+	-- This probably causes taint... hm.
+	local prev_fade_time = UIDROPDOWNMENU_SHOW_TIME
+	UIDROPDOWNMENU_SHOW_TIME = 4
 	local display = GUI:Create("Frame")
 	if _d then
 		display:SetUserData("eoiST",_d)   -- warning! warning! kludge detected!
 	end
 	_d = display
 	display.sizer_s:SetScript("OnMouseUp",nil)
 	display.sizer_e:SetScript("OnMouseDown",nil)
 	display.sizer_e:SetScript("OnMouseUp",nil)
 	]]
 	display:SetCallback("OnClose", function(_display)
+		UIDROPDOWNMENU_SHOW_TIME = prev_fade_time
 		_d = _display:GetUserData("eoiST")
 		self.display = nil
 		GUI:Release(_display)
 		flib.clear()
 		collectgarbage()
 		StaticPopupDialogs["OUROL_SAVE_SAVEAS"].OnAccept (dialog, dialog.data)
 		dialog:Hide()
 	end,]]
 }
+-- Workaround this bug:  http://us.battle.net/wow/en/forum/topic/3278901991
+if true then
+	-- Verbatim copy of UIDropDownMenuTemplates.xml:155 or so, except as
+	-- tagged with CHANGE.
+	local function onenter (self, motion)
+		if ( self.hasArrow ) then
+			local level =  self:GetParent():GetID() + 1;
+			local listFrame = _G["DropDownList"..level];
+			if ( not listFrame or not listFrame:IsShown() or select(2, listFrame:GetPoint()) ~= self ) then
+				ToggleDropDownMenu(self:GetParent():GetID() + 1, self.value, nil, nil, nil, nil, self.menuList, self);
+			end
+		else
+			CloseDropDownMenus(self:GetParent():GetID() + 1);
+		end
+		_G[self:GetName().."Highlight"]:Show();
+		UIDropDownMenu_StopCounting(self:GetParent());
+		if ( self.tooltipTitle ) then
+			if ( self.tooltipOnButton ) then
+				GameTooltip:SetOwner(self, "ANCHOR_RIGHT");
+				GameTooltip:AddLine(self.tooltipTitle, 1.0, 1.0, 1.0);
+				GameTooltip:AddLine(self.tooltipText, nil,nil,nil,1);  -- CHANGE added nil->1 arguments
+				GameTooltip:Show();
+			else
+				GameTooltip_AddNewbieTip(self, self.tooltipTitle, 1.0, 1.0, 1.0, self.tooltipText, 1);
+			end
+		end
+	end
+	-- end verbatime copy
+	for i = 1, UIDROPDOWNMENU_MAXLEVELS do
+		local list = _G["DropDownList"..i]
+		if list then
+			for j = 1, UIDROPDOWNMENU_MAXBUTTONS do
+				local button = _G["DropDownList"..i.."Button"..j]
+				if button then
+					--print("button fixup",i,j)
+					button:SetScript("OnEnter",onenter)
+				end
+			end
+		end
+	end
+end
 -- vim:noet

Mercurial > wow > ouroloot

comparison gui.lua @ 25:cb9635999171