Garan

Thanks for the feedback.

The Opacity really should go to 0, that was an oversight which you can change by editing line 444 and change the 1 to a 0:
self.Options.OpacityScrollBar:SetMinimum(1);
should be:
self.Options.OpacityScrollBar:SetMinimum(0);

Allowing placement at the top of the screen will require changing the layout of the control - the movement tab currently sits above the money display so that the entire wallet is visible as it is moved (as all of the standard controls other than the compass work). Placing the movement tab on the money display will obscur the display during movement but would allow the wallet to move to the top edge of the screen. A compromise would be to change the minimum boundary so that the movement tab can go partway off screen, allowing the wallet to get closer to the edge but always leaving a part of the movement tab accessible.
Changing line 148 from
if newTop<0 then newTop=0 end;
to
if newTop<-19 then newTop=-19 end;

will allow the wallet to get to one pixel below the edge but still allow the user to grab that one pixel border to move it in the future.

Aulderic

Do you plan an adding other currencies that are in the wallet? Regular skirmish marks would be a very nice addition

Garan

Thanks for your interest. Unfortunately, there is no Lua access to any of the other items in the barter wallet.

Adra

tyvm for the responses

I tested your code modifications, the placement at the top is pretty ok, but I can't move it anymore

Maybe placing the drag bar to the left like in "ultra mini packs" could be a good compromise for a majority of users.


For the background transparency it's not the good code.
I'm not looking for the whole plugin invisibility, just its grey translucent background.

Other point, a right click only opens me opacity menu, and I didn't found how to close it

Garan

The background is controlled by line 156:
self.WalletPanel:SetBackColor(Turbine.UI.Color(.3, 0,0,0));
This line is actually a carryover from a prior version before the "move" panel was added and I was experimenting with a bug that was preventing updates. Now that the display elements are on a separate panel of their own, this line is no longer needed and can be commented out (add two dashes to the start of the line) or deleted. Alternately, you can simply set the alpha channel to 0 by changing the line:
self.WalletPanel:SetBackColor(Turbine.UI.Color(0,0 ,0,0));

Allowing the plugin to position off screen does have the undesirable side effect of hiding part of the Options panel, including the Close button.
This is fairly easily fixed by changing line 380 from:
self.Options:SetPosition(self:GetLeft(),self:GetTo p());
to
self.Options:SetPosition(self:GetLeft(),self:GetTo p()+20);
the extra 20 pixels is enough to keep the full options dialog visible.

Adra

Thank you very much !

It works fantasticly great now

You rule !

Vald

Wallet crashes my system if I load it and then reload or unload plugins. This occurs as soon as I change stances on myself. Tested this with multiple plugins and found this to be the problem. This occurs even when Wallet is the only plugin I load (aside from the manager or bootstrap loaders).

Garan

This is not a bug in Wallet. It is a bug in the underlying Turbine implementation which was introduced in the recent patch. This is related to the same bug which people are experiencing when they use the /plugins refresh command and then mount a horse. There are a number of threads already explaining that this is not a problem in any one plugin (any client crash in a managed environment is not the fault of the managed code, but rather the environment itself and in this case there is no workaround which can eliminate these crashes) even if you only experience it with one plugin on your particular system. Anyone experiencing client crashes is urged to report them to Turbine with as much detail as possible about the conditions under which you are able to reproduce the crash.

EDIT: In the case of Wallet, the crashes are related to the LocalPlayer:GetAttributes() method. Accessing this method and then unloading a plugin and performing just about any activity will cause a client crash. If you don't unload or refresh plugins it remains stable. That is the most information and only workaround I can provide until Turbine fixes the :GetAttributes implementation.

Wicked Mouse

Suggestions:
- add more currencies, like the marks, medallions, seals, relic shards....
- add options to tick/untick the various currencies, saved for individual characters.

Wallet plugin seems to load normally with Turbine's new Plugin Manager.

Garan

Thanks for the feedback.

Turbine does not yet expose any of the additional currencies to Lua.

Wicked Mouse

Playing a Creep I get the following error by the way:

Garan

Yeah, that's because Turbine didn't implement the :GetMoney method for Creep characters - you'll find that most of the attribute related plugins won't work for Creepside

Wicked Mouse

Woah! Great update!

Garan

Version 1.02 supports all of the new Barter Wallet currencies and some new ordering/docking functionality to make handling all of the new currencies easier.

Unfortunately, Turbine still has not implemented :GetMoney() for Creeps so Wallet will not work for Creeps until version 1.03 is published (no anticipated date yet).

Note that not all of the currencies have icons, so displaying them is of fairly little value since there's no icon to tell you what the number floating in the middle of your screen is for . If Turbine ever adds icons to those currencies, the icons will automatically be updated.

Adra

Great work !

and what a pity for creeps commendations, I hope Turbine will fix that soon