Auctionator

Auctionator

141M Downloads

Disable pricing tooltip on mutliple attached mailbox items - Fix included

ceylina opened this issue ยท 4 comments

commented

Adding pricing data on multi items attached to a mail message is not a feature some users are happy with. In order to accommodate the needs of our user base, it might be best to add an option in tooltip settings to disable multiple line attachment pricing data.

I will work up a fix if I have the time and post it here

commented

Fix proposal - we add IsModifierKeyDown() so that it does not conflict with dropdown settings for "show disenchanting details" in the tooltip options setting. Should all tooltip pricing and DE data be disabled, this will not show pricing, regardless of modifier key being down.

Without a modifier down, it does not show any information. If they want the names of attachments only, they can install postal.

AucitionatorHints.lua at line 611

hooksecurefunc ( "InboxFrameItem_OnEnter",
function ( self )
local itemCount = select( 8, GetInboxHeaderInfo( self.index ) )
local tooltipEnabled = AUCTIONATOR_V_TIPS == 1 or AUCTIONATOR_A_TIPS == 1 or AUCTIONATOR_D_TIPS == 1

if tooltipEnabled and itemCount and itemCount > 1 and IsModifierKeyDown() then
  for numIndex = 1, ATTACHMENTS_MAX_RECEIVE do
    local name, _, _, num = GetInboxItem( self.index, numIndex )

    if name then
      local attachLink = GetInboxItemLink( self.index, numIndex ) or name

      GameTooltip:AddLine( attachLink )

      if num > 1 then
        Atr_ShowTipWithPricing( GameTooltip, attachLink, num )
      else
        Atr_ShowTipWithPricing( GameTooltip, attachLink )
      end
    end
  end
end

end
);

commented

What are other options here? I hate to disable functionality to avoid conflict with other addons... are there configuration options I can add that would alleviate this problem?

commented

Added toggle for mailbox tooltips.

commented

4.0.8 Released