Inventorian

Inventorian

660k Downloads

Classic Keyring

questionmarkquestionmark opened this issue ยท 8 comments

commented

Today wow classic updated to and added the keyring. Current version of classic inventorian doesn't support this, when you click the keyring it just opens the inventory and not a separate keyring window.

commented

For a quick fix, edit Inventorian.lua and change BAG_CONFIG:bags from:
{ BACKPACK_CONTAINER, 1, 2, 3, 4 }
to
{ KEYRING_CONTAINER, BACKPACK_CONTAINER, 1, 2, 3, 4 }

No idea if this is going to break something else, but I haven't had any issues so far.

commented

Adding the keyring to the main bag window is not ideal, as it would be rather confusing to have an additional 16 slots there which you can't use for normal items.

I was planning to make a new window and add a key button to the bag frame, which I'll likely work on this weekend.

commented

Pretty sure I have this working, is there anyone around to accept a PR?

commented

Any news on this keyring update ?

commented

Check out my fork here, it's pretty much fully working https://github.com/blastehh/Inventorian

commented

This is now available

commented

No news ? :(

commented

Quick and very dirty temporary fix here.
Keyring window shows some redundant information (currency and bag slot) but otherwise works. May have issues manually moving keys around, but right-clicking them should still work.
Probably throws some new errors - you're on your own there.
Can't test with a full keyring to make sure it resizes properly/etc, but I don't expect any problems with that.
Added "/inventorian keys" command, just in case.

Links contain an updated Inventorian.lua (remove .txt. extension if you dl from git). Be sure to back up your original before replacing it.

Inventorian.lua.txt
https://pastebin.com/LRgNrL4F