Kyzer's suggestions

[Chironex]

Instead of creating a new topic each time i propose a suggestion for ulx, i will post them all in this one.

This is only suggestions that i would like to see in a future version


1- In the Client menu, in the list of players, what about adding a "All Players" option, which will run the command we choose with the parameter <all> ? I know about some commands doesn't allow a <all> parameter, maybe a way to make buttons grey or something so we can't press them ?


2- I always think "why they don't log NPCs spawn?"

Code: [Select]

local function NPCSpawn( ply, ent )
ulx.logSpawn( string.format( "%s<%s> spawned NPC %s", ply:Nick(), ply:SteamID(), ent:GetModel() ) )
end
hook.Add( "PlayerSpawnedNPC", "ULXLogNPCSpawn",NPCSpawn, 20 )


[Megiddo]

I thought about #1, but I'd want to add team detection in as well. Also, derma doesn't support "greying out" right now.
Didn't realize we had forgot #2.

[MrPresident]

a way to prevent the use of <all> on commands that don't support it could be a feed back of something like...

'This command doesn't support multiple targets'

This could also be useful if you tried to type it as well.. like.. "!ban <all>" it would print back to you a response rather than the default message which I believe is "No Target Found"

[JamminR]

Hmm. I could have sworn it did have feedback. At least at one time it did.
Perhaps we broke it, or, I dreamed it and this idea is just a confirmation of my dream becoming the future.
Yeah..that's it.

[Megiddo]

Doh, I still haven't done #2...

[Megiddo]

I'm a horrible person who should be shot for still not doing #2. Anyways, sticking it in SVN right now.

[Chironex]

Hello,

A small suggestion: replacing the keyword <all> with a simple *

[JamminR]

I like the asterisk = <all> idea. If not a replacement perhaps, in addition too. I'm not sure how many admins use the multiplayer functionality anyway.
I mean, I know we do, but considering many don't even read the readme's, I'm curious as to how many even know it exist.

[Megiddo]

Would be easy to do, only need to modify one function.

[Chironex]

Thanks, i just saw it has been done in the SVN verison, like, a day after i suggested

[MrPresident]

good suggestions get implemented. 

[JamminR]

good suggestions get implemented. 

And those who make good suggestions get credit
(Check the Ulib_readme change log too )