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

Classic Ironsight is not functional #202

Open
MyGamepedia opened this issue Dec 27, 2024 · 5 comments
Open

Classic Ironsight is not functional #202

MyGamepedia opened this issue Dec 27, 2024 · 5 comments

Comments

@MyGamepedia
Copy link

In SP, you can enable simple zooming (for crossbow and 357) instead of ironsight via cl_ironsight_classic or in main menu options. The simple zooming is forced to be enabled in MP with no option to toggle it. SourceCoop reenabled ironsight, but you still can't toggle this behavior.

29.mp4
@MyGamepedia MyGamepedia changed the title Classic Ironsight is not working Classic Ironsight is not functional Dec 27, 2024
@ampreeT
Copy link
Owner

ampreeT commented Dec 30, 2024

In our implementation, we currently query the ConVar cl_ironsight_classic on weapon switch as we don't get notified when the client changes the value. Could you try retesting this again by setting the value and then switching to a weapon with ironsights?

@Alienmario Is there a better hook/event that you could think of that we can use to query this ConVar?

@Alienmario
Copy link
Collaborator

Haven't tried in BM - OnClientSettingsChanged / AddCommandListener

With command listener, you can catch some quite internal commands.

AddCommandListener(OnClientCommandCustom);
public Action OnClientCommandCustom(int client, const char[] command, int argc)
{
	static char buffer[255];
	GetCmdArgString(buffer, sizeof(buffer));
	PrintToServer("%s %s", command, buffer);
}

Looks like we already use one in the project.

@Alienmario
Copy link
Collaborator

As for catching the change inside client console, don't think we can do it.

@MyGamepedia
Copy link
Author

In our implementation, we currently query the ConVar cl_ironsight_classic on weapon switch as we don't get notified when the client changes the value. Could you try retesting this again by setting the value and then switching to a weapon with ironsights?

It works, but also found that 357 is pretty buggy with cl_ironsight_sticky 1.

30.mp4

@MyGamepedia
Copy link
Author

It works, but also found that 357 is pretty buggy with cl_ironsight_sticky 1.

I think it should be new issue. If it's not seems possible to solve #202, you can close it. But it doesn't behave as expected however, so I would leave it until a solution is found.

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

No branches or pull requests

3 participants