BagSync

BagSync

4M Downloads

Feature request: Highlight current character's numbers

Snufkin2153 opened this issue ยท 6 comments

commented

Hi there.

I have a weird issue. Nothing's broken, but in wide and long tooltips it's sometimes hard to see the amount of an item/currency that the current character has.

There is an option in Display -> Tags to display green โœ“ next to current character name.
But it doesn't really help, as seen on the screenshot below:
image

Could you please consider implementing one (or several) of the following options? Whichever would be easier for you.

  • Add same checkmark on the right side near the number, or an arrow, or some icon, etc;
  • Highlight that number in different color;
  • Add option in Display -> Tooltip Sorting to always put current character first or last in the tooltip;
  • Apply the class colors to the numbers, same as character names. I see that there is an option in Color->Display that kinda fits this description ("Display item totals by character class color"), but it just doesn't do anything, numbers remain plain white. Or have I really misunderstood the purpose of this option?

Thanks in advance!

commented

Hmm I suppose I can put an option to always place the player on the first line. This will only work so long as you don't use Custom Sort Order option in BagSync.

If possible can you please message me on CurseForge Or Wowinterface with your BagSync.lua file found under the WTF folder of your WTF\Account\YOUR ACCOUNT NAME\SavedVariables

I would normally ask you to attach it here, but GitHub doesn't have any privacy settings for posting attachments. You can use https://wormhole.app/ to upload the file temporary if you like, https://filetransfer.io/ is another one

commented

Messaged you on CurseForge.

commented

Messaged you on CurseForge.

Thank you! I'll take a look and see if I can put a simple fix to just slight the active player on top. Using your list makes it easier for me to duplicate your listings so I know it will work properly. I'll work on it sometime today or tomorrow ๐Ÿ˜ƒ

commented

Try this version that I modified and let me know if it works for you. You will need to select the new sorting option under the Display config for BagSync.

BagSync.zip

commented

Seems to be working perfectly, thank you!

commented

Seems to be working perfectly, thank you!

That's great! Thanks for confirming the changes worked. I'll go ahead and fix up the TOC version numbers and then push the changes to live.