arkayenro / arkinventory

A World of Warcraft Inventory mod for Retail, Burning Crusade, and Classic
108 stars 14 forks source link

ArkInventory_Tracking_Reputation - Anglers, Tillers, Alliance Vanguard #1066

Open Calesta opened 6 years ago

Calesta commented 6 years ago

I started playing around with the new reputation tracker that you added a few versions ago.

It seems like you're still working on enhancing the feature because I've noted some new positive changes in the latest build (3.08.06) like no longer showing inactive factions in the selection list.

In the faction list, the Anglers, Tillers, and Alliance Vanguard are all presented in the yellow text of a category header when not selected rather than the white text of the other factions. In addition, they appear separate from the category header instead of falling neatly underneath.

arkinventory_tracking_reputation

I'm sure this is related to how these factions are being treated by Blizzard in the reputation pane as all three appear as subcategories there and they can't be deactivated. The reputation tracker probably just uses the provided Blizzard data to build / display the list.

I hesitate to ask for you to put a cosmetic fix in place to address it because if Blizzard decides to change it later it might cause issues, but I still thought I'd ask to see if there was some sort of elegant solution.

arkayenro commented 6 years ago

i've updated it so that child header reps dont insert a blank line, they will still be yellow but the same size as a normal rep

will be in the next release after 3.08.08 (which is still pending)

joevt commented 6 years ago

One problem with the Blizzard UI is that you can't make inactive the header reps even if their child reps are inactive. Is that something AI can do in it's UI?

arkayenro commented 6 years ago

not easily as i process downwards, meaning when im at the header row i have no idea whats underneath as i havent gotten there yet

arkayenro commented 6 years ago

oh, i think i understand what you mean. you want AI to let you deactivate/activate the reputations and while the child headers wont hide in blizzard they would in AI