Change log

James D. Callahan III:
    - Corrected logic for expansion filter toggle buttons. Fixes ticket # 1018.
    - Renamed expansions in the SavedVariables to match what is now in the revamped GUI code.
    - Category tabs will now remember the state of their contents when switching between them or changing the sort method - currently non-functional for sub-header entries.
    - Standardized arguments to all calls of ListFrame:Update().
    - Moved the OnShow handler for MainPanel into MainPanel:Display() for better initial tab handling.
    - The sorted lists for locations and acquisition types are now processed once, upon creation, instead of every time the list for those tabs is re-initialized.
    - Massive code cleanup/simplification - round 2.
    - Changed the progress bar texture and tweaked the color again.
    - Removed the custom progress bar texture in favor of a default texture, and lightened the color substantially.
    - Fixed issue with collapsing the panel. Fixes ticket #1015.
    - Massive code cleanup/simplification.
    - Standardized the progress bar and exit button - they will no longer resize when the panel is expanded or collapsed.
    - Don't call InitializeFrame() from addon:OnEnable() - call it from MainPanel:Display() if non-nil.
    - MAINPANEL_NORMAL_WIDTH and MAINPANEL_EXPANDED_WIDTH are now MainPanel.normal_width and MainPanel.expanded_width
    - Moved some table definitions into do blocks with the singular functions which used them.
    - Removed some dead code/declarations.
mike:
    - Fixed the rep levels for the Blacksmithing recipes that were mentioned in ticket 1014
James D. Callahan III:
    - The list frame will no longer have its background changed between expanding and collapsing the panel.
    - I broke the tabs. Now, I fix the tabs.
    - Make sure the scroll bar's OnValueChanged script doesn't try to update the list while the list is being initialized. Fixes ticket #1011.
    - Only update the list when selecting a tab after the first time the list is shown, to avoid updating twice.
    - Re-wrote InitializeCheckBox() and renamed it to CreateCheckBox() which simplified a bunch of code and fixed ticket #1013.
    - Fixed logic when clicking tabs to not occur multiple times.
    - Ensure that subheaders and subentries truncate instead of overlapping the scroll bar.
    - Properly aligned the title bar text.
    - Fixed the width of the progress bar in the expanded panel.
    - Solidified the empty portion of the progress bar to hide the underlying texture.
    - Removed the word "Miscellaneous" and the "Display Exclusions" checkbox from the Miscellaneous filter menu.
    - Ensure that the filter checkboxes are properly marked after a reset.
    - Changed the mouse-interactible area of the panel according to its expanded/collapsed state.
    - Changed tooltip anchoring to account for changes to the panel.
    - Finished GUI revamp for expanded panel and removed textures that are no longer in use.
    - Changed the logic for the filter buttons to make the currently-selected filter persist between expanding/collapsing the panel.
    - Made the profession icon line up appropriately with the mode button.
    - Consistency thing - Replaced single quotes with double quotes for the strings which were using them.
    - Revamped GUI to use Blizzard textures. The main panel is wider, fitting more text in the list. The expanded panel is currently unfinished.
    - Removed unused variables and re-sorted some declarations.
mike:
    - document the new slash cmd, /arl tradelinks
    - trim trailing space
    - Added a new slash command to print out the generated tradeskill links. (/arl tradelinks) Suggested by rumpl3d4skin, Ticket 1008
James D. Callahan III:
    - Re-enabled the updater - if the first return from GetTradeSkillLine() is not "UNKNOWN", allow the scan.
    - Added detection for currently-supported TradeSkill AddOns to the mode-button so toggling between tradeskills while the frame is hidden will work.
    - In addon:Scan() - Check for current_prof being "UNKNOWN" instead of checking the scan_button's parent to determine if a tradeskill frame is open.
    - In ListFrame:Initialize() - changed definition of current_tab.
    - Re-organized view-tab code and moved MainPanel settings out of the tab do-block.
mike:
    - Added world drop locations to the rest of the Leatherworking recipes that needed it.  Jewelcrafting: just 69 recipes left to fill in world drop locations for.
James D. Callahan III:
    - Made Tooltip_AddWorldDrop() get the recipe's quality color itself so the quality_color parameter can be dropped from both it and addon:DisplayAcquireData().
    - Yanked code from ListItem_ShowTooltip() to make addon:DisplayAcquireData() so external AddOns can generate their own lists.
    - In ListItem_ShowTooltip() - removed un-used playerFaction variable.
    - It is no longer necessary to nil addon:InitializeLookups() after it has been run - it now does so itself.
    - Renamed addon:InitializeRecipe() to the more proper addon:InitializeProfession()
    - Re-added addon:GetRecipeData(), but without returning the recipe table itself - Usage: ARL:GetRecipeData(spell_id, data)
    - Commented out addon:TRADE_SKILL_UPDATE() and the corresponding updater frame - not only does the event fire for others (without saying who it fires for), but calling addon:Scan() results in it firing, which calls addon:Scan() a second time.
    - In ListFrame:Initialize() - don't check which tab we're viewing, just call it's Initialize().
    - Pulled code from ListFrame:Initialize() to make RecipesTab:Initialize().
    - Moved FormatRecipeText() to be beneath SetTextColor().
    - Disable entry and state buttons in ListFrame:ClearLines(), and enable those that are shown in ListFrame:Update() - fixes issue of highlight texture showing when not appropriate.
    - Converted remaining uses of RecipeMatchesSearch().
    - Split code from ListFrame:Initialize() into AcquisitionTab:Initialize() and LocationTab:Initialize().
    - Renamed RecipeMatchesSearch() to SearchBox:MatchesRecipe()
mike:
    - Fixed recipe binding for "Hard Khorium Battleplate" .. RBOP->RBOE
    - Added some new quests
    - Corrected some phrases
    - Added the dalaran cooking quests to 3 cooking recipes
    - Fixed the quality coloring of some recipes, some were not colored COMMON like they should have been.  Scanner also didn't pick up the ones fixed in this. Also for "Imbued Netherweave Boots", changed AddRecipeVendor() to AddRecipeLimitedVendor()
    - Fixed the quality coloring of trainable only recipes, some were not colored COMMON like they should have been.  Scanner also didn't pick up the ones fixed in this
    - Fixed the quality of many trainable-only recipes in the Alchemy, Enchanting, Leatherworking, and Tailoring databases.  * -> COMMON
    - Fixed the quality of some Inscription recipes
James D. Callahan III:
    - Turned off the manual ChangeLog. Did I mention that having to make a commit to the project to do this is fucking retarded?