Telented shows up in Deucecommander's "others" menu, but the display name there is "options" instead of "Talented".
DeuceCommander is a hack and might lead to such things. I don't think it's either a bug in Talented or DeuceCommander or simply the fact that this particular hack is not suited for my use of AceOption table.
Hi-
I just started using this great mod. I was wondering though is it possible to get a
scale option - It does take up a lot of space. Otherwise very good addon.
I'd love to see the ability to make a template for another class on my hunter so i can send that to someone else. It's not uncommon people ask me how to spec their class but if i'm on my hunter i can't access any template but my hunter's i think?
@Malfeus : Yes, it does take quite a bit of space. The Default map too. I tried to reduce the space using different options and didn't like the result much. I'll make it into an option.
@pheace : Yeah, this is probably the second most asked feature (just after the "my tooltip are funny" one). I'm working on an optional module that would offer those feature. It just take a little bit of time, but my schedule right now is to hit 70.
Telented shows up in Deucecommander's "others" menu, but the display name there is "options" instead of "Talented".
DeuceCommander is a hack and might lead to such things. I don't think it's either a bug in Talented or DeuceCommander or simply the fact that this particular hack is not suited for my use of AceOption table.
Edited: Note to self: be less of an ass, replying to forums before having coffee in the morning is bad.
Add X-Category to your .toc, that'll get it out of others. It will still show up with an odd name, but that can be easily worked around if you want to do so.
Add X-Category to your .toc, that'll get it out of others. It will still show up with an odd name, but that can be easily worked around if you want to do so.
Deal. I'll even make it so that it appears as "Options of Talented" instead of "Options". Putting "Talented" here would be really changing the semantics of this part of the AceOption table. No ETA, as I have some changes already being worked on.
that it'll be posible to export a link to wow's talent site also.
Really ? This is a little bit complex, as US and europe don't have the same links. I dodged the issue when importing by using a regular expression. Also, I could print it on the chat, but it would be useless without Prat or another addon to be able to copy from the chat box.
Doesn't really matter what talent site, and support for 1 site would be plenty, I'd just like to be able to send a talentbuild ingame or over messenger without ever having to open IE :) Ppl having to type out the link because they don't have prat, or some other link copy mod, would be their own problem :)
You can install it and verify that it works, but don't send me the result yet.
Minor bug: it tries to unregister from PLAYER_ENTERING_WORLD every time through it's main event, even though the event can be called multiple times in a run if you ding or spend talent points.
Just needs a IsEventRegistered or whatever the AceEvent call is to protect the unregistration.
You can install it and verify that it works, but don't send me the result yet.
Minor bug: it tries to unregister from PLAYER_ENTERING_WORLD every time through it's main event, even though the event can be called multiple times in a run if you ding or spend talent points.
Just needs a IsEventRegistered or whatever the AceEvent call is to protect the unregistration.
Yup, here's my bugsack just in case:
[2007/01/29 02:21:13-176-x2]: Talented_Data\Core.lua:99: AceEvent-2.0: Cannot unregister event "PLAYER_ENTERING_WORLD". "Talented Data Gatherer" is not registered with it.
First, I think users will find the concept of a "current template" very confusing. I can kinda understand what you mean -- it's like a wrapper for the current talent distribution and works like a template -- but I don't think most people will. A template that works with slightly different rules than other templates is counter-intuitive IMO. (And I don't just say this on a hunch. I started including Talented in my UI and nearly everyone was confused and made posts about that in the beta forum.)
For what it's worth, I'd suggest doing something akin to what SpecialTalent did. You could have two modes: standard mode and template mode. In standard mode, it would behave exactly like the normal talent window, except you see all three trees and get confirmations. These are all easily understoond concepts and behave similar to the blizzard window. The only template-related things available in Standard mode would be switching to template mode and perhaps something to apply a template. I think seeing a "Template Mode" command in the menu would be very clear to a new user; they'll know that the rules change when they change the mode. Of course, in template mode, it would behave like it does now and they can access the more powerful features of your add-on.
I know these suggestions aren't huge functional differences, but I think it would make a world of difference in terms of making talented a lot more intuitive to new users. Right now, I think a lot of people are opening it up and not having any idea how to get going or whether something is bugged. At least that was my experience and the feedback I've gotten.
Oh, and one other small suggestion. When applying a whole temple, I'd either give only one confirmation of have a "Yes to all" option. I don't think anyone would want to go through 61 confirmations to apply a layout. I'm guessing most people just turn that option off and then turn it back on, something that could easily be automated.
I understand that it looks confusing. It is confusing. I did not really though about the details of how it would work. I was looking at the way Blizz did it and I was thinking about doing... diferently. The core of the tree creation is really different from Blizz. That's what I wanted to do, first. I think mine is better, but it's probably a matter of taste :-)
I find your advices and suggestions very interesting. I'll try to do that. I'm 69 so I may be able to have a little time to devote to this addon very soon.
Current diff file size is around 90Kb, so that's pretty much a complete overhaul. Scaling, Data, and almost everything that has been discussed here is in it. It's not yet working fully but a lot of work has been done.
[2007/02/08 03:38:43-863-x28]: Talented-1.0\editmode.lua:75: attempt to perform arithmetic on local 'original' (a nil value)
Talented-1.0\editmode.lua:91: in function `OnTalentClick'
Talented-1.0\ui.lua:338: in function <Interface\AddOns\Talented\ui.lua:337>
2007/02/23 09:08:39-1877-x1]: Talented-1.0\core.lua:339: Babble-Class-2.2: Bad argument #2 to `GetHexColor' (string expected, got nil)
AceAddon-2.0-28672:455: in function <...s\!SurfaceControl\libs\AceAddon-2.0\AceAddon-2.0.lua:432>
<in C code>: ?
AceEvent-2.0-25921:320: in function `TriggerEvent'
AceEvent-2.0-25921:962: in function <...s\!SurfaceControl\libs\AceEvent-2.0\AceEvent-2.0.lua:955>
---
edit: adding babble-class to externals doesnt fix problem :(
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
DeuceCommander is a hack and might lead to such things. I don't think it's either a bug in Talented or DeuceCommander or simply the fact that this particular hack is not suited for my use of AceOption table.
I just started using this great mod. I was wondering though is it possible to get a
scale option - It does take up a lot of space. Otherwise very good addon.
Malfeus
@pheace : Yeah, this is probably the second most asked feature (just after the "my tooltip are funny" one). I'm working on an optional module that would offer those feature. It just take a little bit of time, but my schedule right now is to hit 70.
Edited: Note to self: be less of an ass, replying to forums before having coffee in the morning is bad.
Add X-Category to your .toc, that'll get it out of others. It will still show up with an odd name, but that can be easily worked around if you want to do so.
Deal. I'll even make it so that it appears as "Options of Talented" instead of "Options". Putting "Talented" here would be really changing the semantics of this part of the AceOption table. No ETA, as I have some changes already being worked on.
Really ? This is a little bit complex, as US and europe don't have the same links. I dodged the issue when importing by using a regular expression. Also, I could print it on the chat, but it would be useless without Prat or another addon to be able to copy from the chat box.
Minor bug: it tries to unregister from PLAYER_ENTERING_WORLD every time through it's main event, even though the event can be called multiple times in a run if you ding or spend talent points.
Just needs a IsEventRegistered or whatever the AceEvent call is to protect the unregistration.
Yup, here's my bugsack just in case:
[2007/01/29 02:21:13-176-x2]: Talented_Data\Core.lua:99: AceEvent-2.0: Cannot unregister event "PLAYER_ENTERING_WORLD". "Talented Data Gatherer" is not registered with it.
---
First, I think users will find the concept of a "current template" very confusing. I can kinda understand what you mean -- it's like a wrapper for the current talent distribution and works like a template -- but I don't think most people will. A template that works with slightly different rules than other templates is counter-intuitive IMO. (And I don't just say this on a hunch. I started including Talented in my UI and nearly everyone was confused and made posts about that in the beta forum.)
For what it's worth, I'd suggest doing something akin to what SpecialTalent did. You could have two modes: standard mode and template mode. In standard mode, it would behave exactly like the normal talent window, except you see all three trees and get confirmations. These are all easily understoond concepts and behave similar to the blizzard window. The only template-related things available in Standard mode would be switching to template mode and perhaps something to apply a template. I think seeing a "Template Mode" command in the menu would be very clear to a new user; they'll know that the rules change when they change the mode. Of course, in template mode, it would behave like it does now and they can access the more powerful features of your add-on.
I know these suggestions aren't huge functional differences, but I think it would make a world of difference in terms of making talented a lot more intuitive to new users. Right now, I think a lot of people are opening it up and not having any idea how to get going or whether something is bugged. At least that was my experience and the feedback I've gotten.
Oh, and one other small suggestion. When applying a whole temple, I'd either give only one confirmation of have a "Yes to all" option. I don't think anyone would want to go through 61 confirmations to apply a layout. I'm guessing most people just turn that option off and then turn it back on, something that could easily be automated.
I find your advices and suggestions very interesting. I'll try to do that. I'm 69 so I may be able to have a little time to devote to this addon very soon.
I was wondering if there was any progress on the scale option and or the talent_data?
Thanks in Advance,
Malfeus
Talented-1.0\editmode.lua:91: in function `OnTalentClick'
Talented-1.0\ui.lua:338: in function <Interface\AddOns\Talented\ui.lua:337>
2007/02/23 09:08:39-1877-x1]: Talented-1.0\core.lua:339: Babble-Class-2.2: Bad argument #2 to `GetHexColor' (string expected, got nil)
AceAddon-2.0-28672:455: in function <...s\!SurfaceControl\libs\AceAddon-2.0\AceAddon-2.0.lua:432>
<in C code>: ?
AceEvent-2.0-25921:320: in function `TriggerEvent'
AceEvent-2.0-25921:962: in function <...s\!SurfaceControl\libs\AceEvent-2.0\AceEvent-2.0.lua:955>
---
edit: adding babble-class to externals doesnt fix problem :(