• 0

    posted a message on nQuestLog
    Yeah, I'm adding everything requested here to my to-do list. The CTRL/SHIFT/ALT-click type stuff for the FuBar menu will be extremely easy, probably a 15-minute thing, so I'll add that first. The Soar stuff shouldn't be too hard, but I guess I'll see when I try adding that.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    I'll fill out the form and request SVN access, probably today or this weekend at the latest. So, hopefully I'll be able to apply the changes soon. I'm pretty comfortable with Lua coding now, so I doubt I'll mess anything up...nothing that I can't fix, at least :).
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Quote from Lewzephyr »

    Thank you yet again 1337rx8...
    its working like a charm. FYI, I am not seeing the issues you mentioned about having QuestGivers checked.


    Well, if you have the QuestGivers module enabled, and you click to view the details of a quest that has QuestGivers data, you should notice a slight stutter every 1 second when it refreshes that whole frame to update the QuestGivers coordinates displayed. When you close the details frame, it stops updating and doesn't stutter of course, but it's annoying if you're trying to run somewhere with the details frame open. I just uncheck it, since that info is in the Comments frame under "Starts:", plus you can have it do a Cartographer waypoint from that, which doesn't make the game stutter, updates more quickly, works across zones, and is more useful. QuestGivers requires Cartographer to work anyway, so I may try to change it to use Cartographer waypoints and work properly across zones, for those who want to use it and who aren't using LightHeaded.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Quote from Lewzephyr »

    Something I was noticing yesterday...
    When I click on a quest and it opens up the nQuestlog quest pane, as well as the Lightheaded pane... the ESC button does not close them.
    Would it be possible to have those windows affected by ESC to close?

    Sometimes when I have those panes up and I start getting attacked (by mob or PVP), I use ESC to close the windows quick so i can see the action.
    Otherwise I have to move mouse cursor to the close option, and sometimes click the abort option....

    Your time / insight is appreciated.


    Here you go. The code was already there in DetailsFrame.lua and was just commented out (was probably from QuestsFu and the author of nQuestLog didn't like that behavior). Just download the new DetailsFrame.lua that I've attached and replace the one in the nQuestLog\modules directory and it will hide the details and comments frames when you hit escape. Works perfectly in my testing, but let me know if it causes any problems for you.

    Edit: Whoops, never mind, looks like this breaks hitting escape for the WoW menu, haha. Removing attachment for now. I'll see if I can fix it.

    Edit2: Ok, it should be working now. I've reattached the DetailsFrame.lua file with the new code. Please try it out and let me know how it goes.

    I don't like how the current code works, as it generates a ton of extra load when you're showing the details frame due to unnecessary refreshes every second, causing the game to stutter badly when you have the details frame open. I plan to fix this, maybe this weekend, by getting rid of the refresh event and only doing a refresh/show/hide based on actual events, such as clicking a quest in nQuestLog. It's a minor annoyance, but still annoying and not how it should be.

    Edit: Never mind, this is the QuestGivers module doing this. I highly recommend unchecking QuestGivers in the nQuestLog options, as the Comments frame gives you this functionality plus more and it doesn't cause really bad stuttering like the QuestGivers module does.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Quote from 1337rx8 »

    TO DO

    • Currently beginning work on adding the non-money rewards from MobMap to the comments window. I'm not sure if this will be possible at this point, and it's likely going to be fairly complicated if it is possible. This should complete the MobMap integration.


    DONE (and currently testing). Also added quest difficulty coloring to quests in the LightHeaded Quest Series. Just need to try to filter out invalid MobMap quest data now.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Thanks for the positive feedback. I'm glad the enhancements are working well. Hopefully, someone with SVN access will see this thread and apply the diffs to SVN. :)

    Additionally, I'm working on further enhancements as follows:

    DONE (and currently testing)

    • Show an estimate of the actual XP you will get if you are currently 6 or more levels above the quest level and thus get reduced XP.
      • This is in addition to the display of the full xp, and is in parentheses to the right of it [for example...(75 @ level 21)].
      • This formula is from WowWiki.com, and although not 100% accurate (plus or minus 3% in my tests), it's close enough. After many hours of trying my own formulas, I've come to the conclusion that Blizzard must use multiple formulas or a very complex single formula to calculate this, depending on various factors, and I don't have enough data to determine exactly what those formulas and factors are.
    • Show information from MobMap.
      • This includes the type of quest (solo, group, dungeon, raid, etc.), the faction (Horde, Alliance, or Horde/Alliance), the zone, the monetary reward of the quest (if any), and the additional monetary reward at level 70 instead of XP.
      • I'm using a formula that I came up with to extract the level 70 reward money out of MobMap's total reward and separate this into two figures. Based on my current tests, the formula is 100% accurate.
      • All of this information is also stored at WowHead.com, so if LightHeaded starts exporting that additional data from WowHead's database, I'll get it from LightHeaded instead. Currently, however, LightHeaded does not support this.
    • Add zone name to quest NPC coordinates.
      • Currently, the comments frame uses calls to LightHeaded to list starting or ending NPC coordinates when you click the NPC name. Since this can get confusing with just coordinates and no zone name, I've added the zone name to the left of the coordinates in the comments frame.
      • Additionally, if the zone name is different than your current zone, and you click to track via Cartographer_WayPoints, it will display the name of the zone in parentheses after the NPC name.
    [br]TO DO

    • Currently beginning work on adding the non-money rewards from MobMap to the comments window. I'm not sure if this will be possible at this point, and it's likely going to be fairly complicated if it is possible. This should complete the MobMap integration.
    • Try to filter out invalid MobMap quest data. MobMap always seems to have the valid quest data in its database, but sometimes an invalid quest entry with the same name will take precedence and mess up the MobMap Info display.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Quote from Lewzephyr »

    well, when you say you added... I am assuming you are refering to the Diff in the post just a few up. If so, I have not seen those changes yet as I usualy pull from the SVN so I am eagerly awaiting somone to upload them.
    The feature I like from mob map is the ability to click on the mob that needs to be killed and the area that the mob can be found in will be outlined in cartographer. If thats what you have added in your diff... Im very stoaked about it (ya thats a good thing).

    Good hunting to you.



    Yeah, that's exactly what has been added. Whatever the objective is, if it can be mapped it will. So whether it's a mob, a drop, or whatever, if MobMap can map it it will open the map and identify the location of that objective. Enjoy! (once it's added)

    Just in case it takes a while for this stuff to get into SVN, I'm attaching the full DetailsFrame.lua and Comments.lua files with my modifications. Just replace the files of the same name within your Addons\nQuestLog\Modules folder and you'll have the new functionality.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Quote from Lewzephyr »

    Like how this does interface with Lightheaded... would love to see it do the same for MobMap.

    Also eager to see the changes that 1337rx8 made.


    For the LightHeaded stuff, the display code is basically copied from LightHeaded, modified a bit, and then used. It calls LightHeaded for the data and then displays it using the copied LightHeaded code (which I and others have modified, of course).

    As for MobMap, what else would you like to see besides what I added? This was the big feature for me, but I can look into adding other MobMap functionality if you could list what you'd like to see. Adding quest rewards to the LightHeaded frame is already on my list, and that will probably come from MobMap, I imagine. Just to clarify, I'm not interested in adding anything that LightHeaded already provides. Thanks.
    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    Here is another diff submission for added functionality for the LightHeaded comments frame. Here's a summary of the changes:

    nQuestLog LightHeaded Comments frame
    - Rearranged the layout a bit for a more compact and (IMO) better-looking display.
    - Corrected the "sharable" display so that it's accurate, rather than being completely broken like it is right now with any quest log (this needs to be fixed in LightHeaded).
    - Fixed the display of reputation which wasn't working due to outdated nQuestLog comments frame code.
    - Added ordered quest series list that allows clicking to view each quest in the list within the same frame.

    Please apply to SVN or let me know what I need to do if this isn't the right way to request this. I've been testing this for about 5 days now and it works GREAT! The only quirky thing is that if two or more quests in a quest series have the same name, only the first one is shown when clicked. I plan to try to fix this, but it's really only a minor concern and doesn't generate any bugs or crashes or anything. I will try to add quest rewards if I can, but the stuff I fixed/added was much higher priority for me.

    Posted in: General AddOns
  • 0

    posted a message on nQuestLog
    I've added functionality to click on quest objectives to have them mapped by MobMap. The diff file is attached. It's very little code to achieve such a wonderful integration that nQuestLog was missing. Please merge with SVN or let me know what else I need to do if I haven't done something right here. Thanks!
    Posted in: General AddOns
  • 0

    posted a message on Cartographer Quests
    I have a few feature requests:

    • Allow quests tagged as unavailable to be manually changed to available. Currently, the priority of unavailable in the LUA code prevents this from working, even if you do manually change it to available. This is necessary as some quests are actually available above the [player level + 3] cutoff in the code. For example, I have a level 20 character who can currently accept a level 25 quest and a level 30 quest, but they register as unavailable and my override has no effect on the map icon.
    • If quest level doesn't appear in the tooltip, adding that would probably be pretty simple. I didn't remember it being there, but I can't verify right now as the server is down. This is recorded in the LUA file anyway, so why not. Please ignore this if it's already in the tooltip.
    • This would just be a "neat" feature (really useful though), but it would be cool if the quest exclamation icons for available quests were colored (green, yellow, orange, red) based on their level as compared to the player's level.

    I may try to add these myself and post what I did if no one responds to this within a few weeks. Thanks for this incredibly useful addon, as I have 10 characters and love to do every quest I can (mostly for rep). This addon removes the headache and constant running around checking NPCs for quests I have abandoned to free up space.
    Posted in: Map/Minimap AddOns
  • To post a comment, please or register a new account.