CurseForge.com

Broker_Wallet

1 - Honor points icon

What is the enhancement in mind? How should it look and feel?
Can you replace the coins icon you are currently using to identify honor points to the faction specific PvP icons?

Please provide any additional information below.
I knew that cargoHonor used the faction specific PvP icons, so I looked at its code and altered my local copy of Broker_Wallet to get the same result.

These are the changes I made:
Line 11

-- icon = "Interface\\Icons\\Inv_Misc_Coin_02",
++ icon = "Interface\\AddOns\\Broker_Wallet\\"..UnitFactionGroup("player").."Icon",

Line 46

-- "Interface\\Icons\\Inv_Misc_Coin_02",
++ "Interface\\AddOns\\Broker_Wallet\\"..UnitFactionGroup("player").."Icon",

And I copied over the icons.
AllianceIcon.tga
HordeIcon.tga

User When Change
Krevlorn Mon, 28 Sep 2009 14:54:17 Changed status from Started to Fixed
Krevlorn Sun, 27 Sep 2009 18:20:33 Changed status from New to Started
gamemaster128 Sat, 19 Sep 2009 21:45:55 Changed description:
  These are the changes I made:
  Line 11
+ [code]
  -- icon = "Interface\\Icons\\Inv_Misc_Coin_02",
  ++ icon = "Interface\\AddOns\\Broker_Wallet\\"..UnitFactionGroup("player").."Icon",
+ [/code]
  Line 46
+ [code]
  -- "Interface\\Icons\\Inv_Misc_Coin_02",
  ++ "Interface\\AddOns\\Broker_Wallet\\"..UnitFactionGroup("player").."Icon",
+ [/code]
  And I copied over the icons.
  AllianceIcon.tga
gamemaster128 Sat, 19 Sep 2009 21:44:46 Create

You must login to post a comment. Don't have an account? Register to get one!

  • 1 comment
  • Avatar of Krevlorn Krevlorn Sun, 27 Sep 2009 18:21:06

    I should be able to make this change. Thanks for the tip.

  • 1 comment

Facts

Last updated on
28 Sep 2009
Reported on
19 Sep 2009
Status
Fixed - Developer made requested changes. QA should verify.
Type
Enhancement - A change which is intended to better the project in some way
Priority
Medium - Normal priority.

Reported by

Possible assignees