shpuld/sui-qc

Simple QuakeC (MenuQC) UI system for FTEQW

sui-qc's Issues

[Suggestion] Avoid FTE-specific __inout functionality

Most of the SUI lib is engine and compiler agnostic (bar a few extensions), but the heavy use of __inout makes your library difficult to port to other setups.
While it certainly can be worked around on the user side, I'd like to suggest changing them to something more compatible, thanks!

sui_pre_draw() and sui_draw() undocumented

CSQC_UpdateView has commented-out references to sui_pre_draw and sui_draw but these functions do not exist. It's unclear to be if the user is supposed to create them.

sui_action_element will throw an error if these functions are not called.

It appears sui_pre_draw was renamed to sui_begin, which sets the variables the rest of the SUI system is checking.

Key binding fails when using FTEQW-SDL2

When using fteqw-sdl2 key press events are sent twice; the first time chary contains the keycode, the second time scanx contains the keycode. This means key binding fails because sui-qc only uses the scanx value.

I worked around this by updating sui_input_event() with:

	case IE_KEYUP:
		if (_sui_binding_command != "")
		{
			local float keycode = (scanx != 0) ? scanx : chary;
			_sui_do_keybind(keycode, _sui_binding_command);
			return TRUE;
		}