Results: 158Comments by: bsmorgan
File: Skillet07-20-16
Skillet is very badly broken in 7.0...
Posted By: bsmorgan
Skillet is very badly broken in 7.0.3 as Blizzard has completely reworked the trade skills interface. Work is in progress to determine if it can be fixed.
File: Breakables06-28-15
Re: Any Plans for 6.2 milling changes?
Posted By: bsmorgan
Also, could you make the tooltip update when a new item is on the button when the mouse is just hovering over the button? I'd even accept not having a tooltip rather than a stale one. The changes show this as implemented but it isn't working for me. I place my mouse over the button and it stays there, clicking each time the icon c...
File: Breakables06-25-15
Any Plans for 6.2 milling changes?
Posted By: bsmorgan
If you do have plans, I'd like to make a suggestion. For both milling and disenchanting, could the button(s) have some visual indication when they are ready (or when they are not ready) to be pressed? I'm thinking dim when not ready (i.e. still processing the last press and or updating to the next item). Also, could you make the t...
File: Skillet06-25-15
Re: recipes not showing
Posted By: bsmorgan
i've added to leatherworking some draenor patterns that are not showing up in Skillet, if i disable it and use the blizzard ui it's all working but in Skillet i cannot find them. Try Skillet again (after using the Blizzard UI). If necessary, close Skillet, type "/skillet flushrecipedata", and open Skillet again. If it still doesn'...
File: Skillet06-11-15
Skillet version 3 has been withdrawn
Posted By: bsmorgan
Due to a large number of issues which I can't reproduce (and therefore can't fix), I am reverting to 2.80 as the latest release version of Skillet. Edit: Progress continues with 2.81, 2.82, etc.
File: Daily Scrapmetal quest05-16-15
I added another slash command, "/db...
Posted By: bsmorgan
I added another slash command, "/dbd reward" (initial state is false to maintain previous behavior). This toggles automatically accepting the quest reward based on the follower average armor/weapon iLevel as calculated by Altoholic. Thanks to maven79 for the idea. If you don't have (or want) the addon Altoholic, then you have...
File: Daily Scrapmetal quest05-16-15
I added the slash commands "/dbd di...
Posted By: bsmorgan
I added the slash commands "/dbd disable" and "/dbd enable". The state is saved per character. All of the followers on one of my characters are at 675/675 so I don't need to do this quest on him atm. I thought this was more convenient than turning off the addon completely on each character.
File: Daily Scrapmetal quest04-22-15
Altoholic version 6.1.005 contains...
Posted By: bsmorgan
Altoholic version 6.1.005 contains the code necessary to activate the code in version 0.6 which displays (in chat) the average Weapon and Armor iLevel of your level 100 followers.
File: Daily Scrapmetal quest04-17-15
Version 0.6 contains cosmetic chang...
Posted By: bsmorgan
Version 0.6 contains cosmetic changes only. The indentation (tabs and spaces) was cleaned up.
File: Daily Scrapmetal quest04-13-15
Below is a patch to use DataStore_G...
Posted By: bsmorgan
Below is a patch to use DataStore_Garrisons (patch submitted and should be included in Altoholic) to help decide which reward would be better (marked with an "*" in the chat output). I include it here at this time because it also fixes a global leak of quest_id and it should not generate any errors if Altoholic isn't installed or...
File: Daily Scrapmetal quest04-05-15
Please consider including the versi...
Posted By: bsmorgan
Please consider including the version number in the filename, i.e. DBD-0.2.zip. I don't like to overwrite old versions until newer versions have "proven" themselves.
File: Daily Scrapmetal quest04-03-15
Now if you really want to get fancy...
Posted By: bsmorgan
Now if you really want to get fancy, add code to parse the player's follower list and compute the average weapon iLvl and average armor iLvl of the active followers. display the results when you interact with these NPCs. This should help with the decision process on which quest reward should be chosen. The addon OneChoice has code...
File: Daily Scrapmetal quest04-03-15
I'd suggest that you use UnitGUID("...
Posted By: bsmorgan
I'd suggest that you use UnitGUID("target") instead of UnitName("target"). This should work in any locale. From my client, http://www.wowhead.com/npc=77377/kristen-stoneforge has a GUID of "Creature-0-3019-1159-30357-77377-00001EBB12" which when properly parsed (see below), the 77377 should be a constant for her. http://www.wow...
File: Gary02-08-15
The buildingUI module never stays o...
Posted By: bsmorgan
The buildingUI module never stays on and always disabled when game loaded. Have to manually enable it. Same problem here, and I don't care about the other modules.
File: Gary02-06-15
I get the following error when I tu...
Posted By: bsmorgan
I get the following error when I turn on the "modules buildingUi" for the first time. Also, if you need to use slash commands for configuration, please accept any case (i.e. lower case the input and compare against that). 2x Gary\BuildingUi.lua:81: attempt to index upvalue 'btnAssign' (a nil value) Gary\BuildingUi.lua:81: in func...
File: Bulk Buy01-29-15
The garrison trading post contains...
Posted By: bsmorgan
The garrison trading post contains a vendor that sells various crafting reagents using garrison resources for currency. With most other vendors, you can shift-click on an item to specify a quantity to buy. The items sold by this vendor do not respond to a "shift-click". Can BulkBuy coerce this vendor into accepting a quantity?
File: ModMod01-26-15
There was an old addon, Bindings Re...
Posted By: bsmorgan
There was an old addon, Bindings Reminder, that was very useful in determining what the existing keybindings were. Would it be possible to add this functionality into ModMod?
File: BodyguardHealth01-24-15
Version 2.0 allows targeting the bo...
Posted By: bsmorgan
Version 2.0 allows targeting the bodyguard through the frame.
File: BulkOrder - Work Orders Automated01-12-15
Can the WorkOrders Reminder frame b...
Posted By: bsmorgan
Can the WorkOrders Reminder frame be made optional, please?
File: Skillet12-18-14
Re: Minor Errors
Posted By: bsmorgan
Sometimes when switching from Tailoring to Enchanting using the top buttons (above the skill bar), it will kick me into Blizzard's UI for crafting. I've not had it occurred when switching from Enchanting > Tailoring, only Tailoring > Enchanting. This is fixed in 2.72.
File: Skillet12-14-14
Re: Minor Errors
Posted By: bsmorgan
Sometimes when switching from Tailoring to Enchanting using the top buttons (above the skill bar), it will kick me into Blizzard's UI for crafting. I've not had it occurred when switching from Enchanting > Tailoring, only Tailoring > Enchanting. I've never seen (or heard) of this behavior and I don't have a Tailor/Enchanter to see...
File: TomTom12-13-14
Can you tell me which version is ne...
Posted By: bsmorgan
Can you tell me which version is newer, TomTom-v60000-1.1.1-beta or the TomTom-v60000-1.1.0 you referenced yesterday.
File: BulkOrder - Work Orders Automated12-13-14
There's no way to do that at the mo...
Posted By: bsmorgan
There's no way to do that at the moment. I'm thinking of adding it to the options. I'm confused. If 1.03 added a "Start All" button to the window, do you still have to left click when you open the window to keep it from auto queuing?
File: BulkOrder - Work Orders Automated12-09-14
Before v1.01 I considered doing an...
Posted By: bsmorgan
Before v1.01 I considered doing an opposite interface, where you shift+click either the NPC or the Start Work Order button to queue all. Hmmm... It is not that difficult to add another button to the existing Blizzard frame instead of a modifier when clicking on the existing button or opening the frame. IMO, this gives the user the...
File: BulkOrder - Work Orders Automated12-08-14
If I have an even multiple of 5, th...
Posted By: bsmorgan
If I have an even multiple of 5, then I want to stop one short of all possible work orders (i.e. always leave my bag slot occupied). Thanks for thinking about it.