DennisRas / WeeklyKnowledge

Track your profession knowledge across your characters
https://www.curseforge.com/wow/addons/weeklyknowledge
6 stars 0 forks source link

Feature request: broker option #29

Open hollo6 opened 2 months ago

hollo6 commented 2 months ago

Hey,

It would be nice to have a broker integration option, like an icon (or even better, a short text that shows the current character's progress in a compressed format) to hover over or click to view/open the grid. I'm thinking similar to how SavedInstances works. My minimap has no room for more buttons and sometimes it's easier to click than type.

FireAllianceNX commented 2 months ago

There currently is already broker integration with an icon. You can turn off the minimap icon with /wk minimap though a mention of that in the README would be helpful.

DennisRas commented 2 months ago

There currently is already broker integration with an icon. You can turn off the minimap icon with /wk minimap though a mention of that in the README would be helpful.

It is in the Readme :)

Hey,

It would be nice to have a broker integration option, like an icon (or even better, a short text that shows the current character's progress in a compressed format) to hover over or click to view/open the grid. I'm thinking similar to how SavedInstances works. My minimap has no room for more buttons and sometimes it's easier to click than type.

As soon as I've made the new #8 feature I guess I could use that data for the broker tooltip. Don't you already have broker data showing? I guess I'll need a data broker addon to see what shows and what doesn't, so I can fix/add some things for those :-)

hollo6 commented 2 months ago

I know how to turn off the minimap icon, I just didn't know the broker existed because there's no mention of it on the addon page so I never looked for it. (I only mentioned the minimap icon to justify why I wanted something clickable that's not it.) So it actually shows up in the broker but definitely could use some improvement. Currently it looks like this: image

And it should be much smaller and with useful info instead (like 1/1/2/1 for treatise/quest/treasure/DMF with colored numbers).

The perfect solution in my mind looks like this:

  1. broker text with numbers only (red for 0, yellow for incomplete, green for complete subcategories) e.g. 1/1/2/1
  2. tooltip for hover-over with current character's progress with details
  3. full window (the one that shows now) on click
hollo6 commented 1 month ago

If this won't be implemented, would it be possible to at least have the broker name shortened so it wouldn't take up so much space? Just show it as WK for example.

NoShotz commented 2 weeks ago

If this won't be implemented, would be be possible to at least have the broker name shortened so it wouldn't take up so much space? Just show it as WK for example.

I mean, most broker bars have an option to hide the name of the broker, which basically does the same thing.

hollo6 commented 2 weeks ago

I mean, most broker bars have an option to hide the name of the broker, which basically does the same thing.

Thanks for the suggestion, this indeed makes it smaller but I would still like a few letters because the icon by itself is too tiny (hard to click on that precise area). Also I'm old and can't, for the life of me, get used to rely on drawings only without words or at least some letters.

And I was kinda hoping for a yes/no/maybe answer so I would at least know what to expect. :)