Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug with colored menus in Half-Life #553

Closed
OciXCrom opened this issue Sep 6, 2018 · 12 comments
Closed

Bug with colored menus in Half-Life #553

OciXCrom opened this issue Sep 6, 2018 · 12 comments
Labels

Comments

@OciXCrom
Copy link
Contributor

OciXCrom commented Sep 6, 2018

I got a report by a user that runs a Half-Life server that when he updated to AMXX 1.9 this happened with all the menus:

If I'm not mistaken, colored menus are not supported in Half-Life. It's probably a mistake made in PR #143. He has been using 1.8.2 before that and the menus worked fine without displaying the color symbols.

I tested as well to make sure - the color symbols appear in menus when using 1.9.0, downgrading to 1.8.2 removes them.

I also noticed another bug while testing the menus. If you leave a menu open and you leave the server, when you come back the menu stays open but you're not able to select anything from it. It happens even if you restart the server.

@Arkshine
Copy link
Member

Arkshine commented Sep 6, 2018

Then 'valve' should be removed from the list.
What about tfc and dmc?

@OciXCrom
Copy link
Contributor Author

OciXCrom commented Sep 6, 2018

@Arkshine Yup, if it really doesn't support colored menus, better remove it ASAP because all HL menus are messed up at the moment.

I don't have those other games so I can't really test.

@Arkshine
Copy link
Member

Arkshine commented Sep 6, 2018

Tested with HL and TFC, and I see the color:

image

@WPMGPRoSToTeMa
Copy link
Contributor

WPMGPRoSToTeMa commented Sep 6, 2018

dmc supports them https://github.com/ValveSoftware/halflife/blob/5d761709a31ce1e71488f2668321de05f791b405/dmc/cl_dll/menu.cpp#L82-L125.
There is no support for colored menus in old hlsdk, so looks like it was added in 6xxx builds for half-life and similar games.

@OciXCrom
Copy link
Contributor Author

OciXCrom commented Sep 7, 2018

I guess we need to check if the build version is 6xxx before adding these games in the list that supports colored menus.

@rsKliPPy
Copy link
Contributor

rsKliPPy commented Sep 7, 2018

AMXX only supports the latest build afaik, there's no need for such check.

@Arkshine
Copy link
Member

Arkshine commented Sep 7, 2018

We are not going to start to adjust stuff based on game build.
There is no reason to not update his server especially when the latest builds contains security fixes.

@WPMGPRoSToTeMa
Copy link
Contributor

WPMGPRoSToTeMa commented Sep 7, 2018

@Arkshine, client build is implied, but anyway AMX Mod X supports only the latest server/client engine build.

@Arkshine
Copy link
Member

Arkshine commented Sep 7, 2018

Yes, you're right.

@OciXCrom
Copy link
Contributor Author

OciXCrom commented Sep 7, 2018

I see, the server that I tested on hasn't been updated for ages. This may be a problem who people that don't know how to update their server, but they will learn eventually.

PS: why did they have to hardcode the colors like that, RGB menus would have been much nicer. 😞

@Arkshine
Copy link
Member

Arkshine commented Sep 7, 2018

As stated WPMGPRoSToTeMa, this is a client-side feature, so the server is unrelated here.

tmp64 added a commit to tmp64/BugfixedHL that referenced this issue Sep 21, 2018
Fixes alliedmodders/amxmodx#553

TODO: Add support for color codes
@AoiKagase
Copy link

AoiKagase commented Oct 18, 2021

Left CS/Right Svencoop
image
It probably doesn't support the color specification of the number, but
It seems that the last color specification of the previous line remains until the next line.

I can change it back to white by specifying \w at the beginning of the menu, but you cannot specify it for the number part.

Is this a client-side bug?
(Svencoop team replied that the menu is implemented differently from other mods, including CS.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants