Compact Vendor

Compact Vendor

20.6k Downloads

Enhancement: Compatible with BuyEmAll

mLgz0rn opened this issue ยท 5 comments

commented

Would love if this would become compatible with the BuyEmAll addon

https://www.curseforge.com/wow/addons/buyemall

commented

It seems written in a way that should be easier to support.

Note to self that the BuyEmAll addon has BuyEmAll:MerchantItemButton_OnModifiedClick(frame, button) method that we can call and provide a frame and make sure that it has the properties it expects and it should display its own dialog properly.

commented

Really would like this to work with BuyEmAll.

commented

Hey @Vladinator. As the author of the BuyEmAll addon, I would like to thank you for adding support for BuyEmAll to CompactVendor. I also had a cursory look at adding support for CompactVendor from BuyEmAll's code but that didn't seem quite as easy to do.

Currently the BuyEmAll addon is written primarily to work with Blizzard's own vendor frames, and from experiences these frame properties can change with a future update. This means the compatibility between CompactVendor and BuyEmAll could break in the future with a new WoW update using this approach of calling MerchantItemButton_OnModifiedClick.

If you want to, I'm totally down to collaborate with you on a sustainable way to maintain support between our addons. I could introduce an API with a stable interface to avoid breakage in the future in a way that works for you as well.

Love to hear your toughts on this.

commented

Thank you so much for adding support for this :-)

commented

Hi @kaelonR, I think it's a great idea. It would allow any vendor related addon to easily integrate with the UI of BuyEmAll.

Since the only thing needed to be passed right now is the index of the vendor item in question, along with an optional anchor preference where the frame should appear. The only last thing that I can imagine is a way to know if the frame is already visible, and what it is currently attached to (anchored to?)

I believe these should be enough for anyone to integrate with BuyEmAll for vendor purchases.