Download
(15Kb)
Download
Updated: 06-21-08 09:56 PM
Pictures
File Info
Updated:06-21-08 09:56 PM
Created:unknown
Downloads:2,342
Favorites:10
MD5:

YouWereKilledBy

Version: 1.0
by: lifetapt [More]

About YouWereKilledBy
YouWereKilledBy is a very small, simple addon that displays who landed the killing blow on you when you die in PvP combat. Upon death, a red "You were killed by Lolmoonfire" message will appear near the top of the screen.

The addon currently does not have any settings, you just install it and that's it. I may add some settings to it in the future, but for now I wanted this to be a very simple and tiny addon.

The code itself is an excerpt from the upcoming version of my other project, Paranoia Enemy Player Alert. Please check it out.

Go ahead and post suggestions/feedback/requests in the comments section, I'll try to reply to all of them.

Optional Files (0)


Post A Reply Comment Options
Unread 06-22-08, 06:29 AM  
VagrantEsha
Token Werewolf Fan
 
VagrantEsha's Avatar
Premium Member
AddOn Author - Click to view AddOns

Forum posts: 27
File comments: 238
Uploads: 38
I like small mods, so this one interests me. Just a couple of thoughts, though...

1. If you do include options, the option to output to the Chat Frame would be nice.

2. As above, if you do include options, you could take the tekkub route and keep them as variables in the file itself to keep the filesize small. I call this "The Tekkub Method", and usually it works quite well.

3. I see you're creating your own font-string, but it might make your mod lighter if you were to use the Blizzard Floating Combat Text stuffs instead, thus removing the need to create another fontstring. You can actually make the text not scroll too, and even stick on screen for a bit, by having it act in its "critical strike" behaviour.

Here's an example of that: CombatText_AddMessage("You were killed by "..name, CombatText_StandardScroll, 1.0, 0.4, 0.4, "crit", false)
Report comment to moderator  
Reply With Quote
Unread 06-22-08, 03:46 PM  
lifetapt
A Murloc Raider
 
lifetapt's Avatar
AddOn Author - Click to view AddOns

Forum posts: 6
File comments: 30
Uploads: 5
Originally posted by VagrantEsha
I like small mods, so this one interests me. Just a couple of thoughts, though...

1. If you do include options, the option to output to the Chat Frame would be nice.

2. As above, if you do include options, you could take the tekkub route and keep them as variables in the file itself to keep the filesize small. I call this "The Tekkub Method", and usually it works quite well.

3. I see you're creating your own font-string, but it might make your mod lighter if you were to use the Blizzard Floating Combat Text stuffs instead, thus removing the need to create another fontstring. You can actually make the text not scroll too, and even stick on screen for a bit, by having it act in its "critical strike" behaviour.

Here's an example of that: CombatText_AddMessage("You were killed by "..name, CombatText_StandardScroll, 1.0, 0.4, 0.4, "crit", false)
Hi, thanks for the comment!
Regarding options, I do plan on adding the basic stuff like output to X frame, change the position of the message, etc. But I'm not quite sure what you mean by "The Tekkub Method" of storing variables in the file? What I would have done was made a preferences table, stored the settings in there, and then specified the table as a SavedVariable in the TOC file. I don't really know of any other way to save preferences, and Tekkub has so many addons that I can't just open one up and see what you mean.
Also, I tried out the floating combat text, but I don't really like it for multiple reasons, the most important of which being that FCT must be enabled for me to send messages to it. Otherwise, calling CombatText_AddMessage throws an error. Also, the "crit" message style is looks very ugly and aliased probably because of the scaling being applied to it. I would likely add FCT as an option though.

Thanks again for the comment
Last edited by lifetapt : 06-22-08 at 03:46 PM.
Report comment to moderator  
Reply With Quote
Unread 06-23-08, 07:34 AM  
VagrantEsha
Token Werewolf Fan
 
VagrantEsha's Avatar
Premium Member
AddOn Author - Click to view AddOns

Forum posts: 27
File comments: 238
Uploads: 38
I can understand you not liking the appearance of the FCT, and to be honest, that's perfectly fair.

As for the options thought, basically it'd involve doing something like this...

Code:
local option_DefaultOutput = true -- Change true to nil to disable the floating text output.
local option_OutputToChatFrame = nil -- Change nil to true to output to the chat frame.

...addon code...

if option_OututToChatFrame then
   DEFAULT_CHAT_FRAME:AddMessage(...blah...)
end

if option_DefaultOutput then
   ...usual output frame code...
end
It wouldn't be exactly like that, of course, but it'd be similar.

If you want ease of use, then using SavedVariables and a slash-command / GUI can be the way to go. But if you're looking to be ultra-minimalist, both of htose can add unnecessary overhead to your mod. It's all about the direction you're taking it in, so you'll know what you want to do.

I just thought I'd make the suggestion really as you seem quite interested in keeping it as small as possible.
Report comment to moderator  
Reply With Quote
Post A Reply



Category Jump: