kstange / MerchantPlus

This addon provides a modern update to merchant (vendor) interfaces in WoW
MIT License
4 stars 0 forks source link

Scrap and Merchant Plus Tab in the Same Spot #5

Closed wheatbread closed 4 months ago

wheatbread commented 1 year ago

Hi, thank you for this great addon!

Came to report that Scrap and Merchant Plus tab are in the same spot.

https://i.imgur.com/MGPvoqD.png

kstange commented 1 year ago

There are a couple different issues here that I can see.

Both addons add tabs to the window in a similar way because if we try to put them into the tab group with the real merchant frame it will cause taint (addon blocked) errors, but we don't really have a way to detect that other addons are doing the same thing.

There's also an issue with switching between the Scrap and Merchant Plus tabs where the window layout won't get updated properly. This similarly a case of not realizing that the other addon exists.

I will look into both issues, but I can't promise anything.

kstange commented 1 year ago

I took a look at this and the author of Scrap actually created a library for managing custom tabs separate from their addon. My solution works in a vacuum, but this should provide a true solution to the problem. I'll try to adapt Merchant Plus to use this library soon.

tusharsaxena commented 1 year ago

Had the same issue, my workaround was to use Dejunk instead :)

kstange commented 4 months ago

I finally got around to trying to implement this, but library used by Scrap is actually incomplete, contrary to its own documentation, and I can't accomplish what I need to be able to do to use it, so I'm not going to be able to do this for now.