1 - Please hook the Skillet tradeskill window too.

What does the provided patch do?

The following trivial change enables ZA to work with Skillet-clone from lilsparky. I used this on QA and submitted a patch for it but it never went anywhere.

+   local hookframe = TradeSkillFrame
+   if Skillet and Skillet:IsActive() then
+       hookframe = SkilletFrame
+   end

And replace all uses of TradeSkillFrame with hookframe.

The cosmetics aren't perfect, but it really doesn't matter. The hook buttons are a few pixels off relative to where they'd be on standard tradeskill frames but that is purely cosmetic.

I use the QA (and hopefully ZA) tradeskill queue for some crafts, and other addons with Skillet (Kevtool queue etc). I can't easily give up on Skillet.

I'm happy to maintain the patch locally, but I figured it might be useful for others and is not disruptive in any way.

User	When	Change
AsaAyers	Oct 17, 2010 at 17:01 UTC	Changed status from Waiting to Replied
Zerotorescue	Oct 13, 2010 at 07:37 UTC	Changed status from New to Waiting
peterwemm	Oct 12, 2010 at 20:30 UTC	Create

You must login to post a comment. Don't have an account? Register to get one!

3 comments

- Reply
- #3
nynjamonkii Oct 28, 2010 at 02:22 UTC - 0 likes
Skillet was updated for 4.0.1 by yossa, and is available on Curse.
- Reply
- #2
Jiminimonka Oct 19, 2010 at 09:29 UTC - 0 likes
Skillet is now discontinued and won't be updated. Lil'sparky changed from Skillet to Gnomeworks (on wowace.com) which is the new replacement for Skillet (and even pre-release is better than Skillet).

The latest version of KTQ also now works with GnomeWorks.
- Reply
- #1
AsaAyers Oct 17, 2010 at 17:01 UTC - 0 likes
Has someone updated Skillet to work with 4.0? Lilsparky hasn't updated it. If you have a 4.0 compatible version of Skillet I'd be interested in getting a copy.

http://www.wowace.com/addons/skillet/repositories/lilsparkys-clone/files/