[OciXCrom]

Since we'll have client_print_color, the show_activity functions will be kinda outdaded because they are hardcoded to use the regular client_print function.

I want to add colors to these functions and sumbit a pull request, but I need suggestions on what is the best method to do this. I mean, I can easily copy/paste the functions and change the print function, but it doesn't seem right. Adding a bool in the current ones will break backwards compatibility (unless it's possible to add a parameter after the "any:..." one). Maybe some #define magic? Any suggestions?

[klippy]

Chat colors are really game dependent as far as I know.

[NiHiLaNTh]

Don't forget that there are other games apart from CS/CZ. AFAIK HL for example doesn't support custom chat colors.

[fysiks]

Since not all games supported by AMX Mod X support chat colors, you cannot change the core functionality to include it.

This is better left for custom modification.

[OciXCrom]

I didn't plan to change it. I wanted to add those functions so we can properly use colorchat with amx_show_activity. client_print_color already is in the core - probably only active when the game supports it.

[fysiks]

Regardless of whether or not a game supports color, a standard notification doesn't need to have any special coloring since which colors and where is extremely subjective. Hence, why this type of thing should be left to individual server customization (that is why the plugin concept exists).

[PRoSToTeM@]

Originally Posted by KliPPy Chat colors are really game dependent as far as I know.

Yeah, and color codes (and its colors) are different.

[OciXCrom]

Originally Posted by fysiks Regardless of whether or not a game supports color, a standard notification doesn't need to have any special coloring since which colors and where is extremely subjective. Hence, why this type of thing should be left to individual server customization (that is why the plugin concept exists).

Like I said - I'm aware of that. Currently you can't add colors to show_activity in CS. If someone wants to do it - he will have to edit the .inc file or add new stocks that will support color because AMXX doesn't offer any function to deal with that. Is adding another .inc file for cstrike that will contain colored show_activity functions a valid solution?

[PRoSToTeM@]

Originally Posted by OciXCrom Like I said - I'm aware of that. Currently you can't add colors to show_activity in CS. If someone wants to do it - he will have to edit the .inc file or add new stocks that will support color because AMXX doesn't offer any function to deal with that. Is adding another .inc file for cstrike that will contain colored show_activity functions a valid solution?

I understood you. You want to replace client_print with client_print_color and you don't want to replace the strings. And after that users can just edit .txt translations to add colors.

[OciXCrom]

Yeah, just copy the functions and use client_print_color instead. If an amxmisc_cstrike.inc file can be included automatically when amxmisc.inc is included, it will be great.