Scroll fights me in a menu full of rows and label/button pairs.

[StrikerTheHedgefox]

Video: https://shadowmavericks.com/files/ShareX/tfury_2025-11-02_15-13-42.mp4

Every time I try to scroll my binds menu up and down, the scrollbar and mouse all fight me, preventing proper scrolling. Can only scroll through it with the keyboard.

void UIFunc_OnConfigureBinds(nk_console* button, void* user_data)
{
	TFURY_UNREFERENCED_PARAMETER(button);

	nk_console_free_children(configureBinds);
	
	// Gather all input types, and form rows in the configure binds menu.
	for (int action = 1; action < INPUT_MAX_INPUTS; action++)
	{
		nk_console* row = nk_console_row_begin(configureBinds);

		nk_console_label(row, gameInputs[action].friendlyName);

		free(inputMenuList[action].keys);
		inputMenuList[action].keys = calloc(1, sizeof(char));

		uint8_t first = 1;
		for (int key = SDL_SCANCODE_A; key < SDL_NUM_SCANCODES; key++)
		{			
			if (keyBinds[key] == action)
			{
				if (!first)
					inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, ", ");

				inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, SDL_GetKeyName(SDL_GetKeyFromScancode(key)));
				
				first = 0;
			}
		}

		for (int joyButton = 0; joyButton < MAX_BUTTONS; joyButton++)
		{
			if (joyBinds[joyButton] == action)
			{
				if (!first)
					inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, ", ");

				char buffer[80];
				snprintf(buffer, sizeof(buffer), "BTN %d", joyButton);
				inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, buffer);

				first = 0;
			}

			if (joyModBinds[joyButton] == action)
			{
				if (!first)
					inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, ", ");

				char buffer[80];
				snprintf(buffer, sizeof(buffer), "Mod. BTN %d", joyButton);
				inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, buffer);

				first = 0;
			}
		}

		for (int hatDir = 0; hatDir < HAT_NUMDIRECTIONS; hatDir++)
		{
			if (hatActions[hatDir] == action)
			{
				if (!first)
					inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, ", ");

				inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, hatNames[hatDir]);

				first = 0;
			}

			if (hatModifierActions[hatDir] == action)
			{
				if (!first)
					inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, ", ");

				char buffer[80];
				snprintf(buffer, sizeof(buffer), "Mod. %s", hatNames[hatDir]);
				inputMenuList[action].keys = appendStrRealloc(inputMenuList[action].keys, buffer);

				first = 0;
			}
		}

		// Slap down a button that'll send us where we want to go. (Bind key menu)
		inputMenuList[action].menuItem = nk_console_button(row, inputMenuList[action].keys);
		nk_console_add_event_handler(inputMenuList[action].menuItem, NK_CONSOLE_EVENT_CLICKED, UIFunc_OnBindKey, (void*)action, NULL);

		nk_console_row_end(row);
	}

	if((int)user_data == 0)
		nk_console_set_active_parent(configureBinds);
	else
	{
		// HACK. Partial reimplementation of nk_console_set_active_parent without scroll set to avoid an assert.
		nk_console* top = nk_console_get_top(configureBinds);
		if (top == NULL)
			return;

		nk_console_top_data* data = (nk_console_top_data*)top->data;
		data->active_parent = configureBinds;
	}
}

[RobLoach]

That makes sense, because it automatically scrolls to make the active component visible. Perhaps a way for us to handle this is to change the actively selected component when scrolling?

[StrikerTheHedgefox]

Yeah, that seems like it would be a bit better, have it clip the selected item to the bounds, so if our selected item hits the top, it moves down, if it hits the bottom, move up.

[StrikerTheHedgefox]

Though, that won't cover scenarios like this one: https://shadowmavericks.com/files/ShareX/tfury_2025-11-03_02-12-26.mp4

So maybe best to only snap the scroll when you're actively pointing at an element, or changing with the arrow keys/dpad.

[StrikerTheHedgefox]

Whenever you find time, would you be willing to look into this one? I'm not familiar enough with nuklear & nuklear_console's code to know how to tackle this issue.

[RobLoach]

I think it has to do with this...
https://github.com/RobLoach/nuklear_console/blob/master/nuklear_console.h#L509-L514

// Scroll to the active widget if needed.
struct nk_rect content_region = nk_window_get_content_region(widget->ctx);
nk_uint offsetx, offsety;
nk_window_get_scroll(widget->ctx, &offsetx, &offsety);
if (bounds.y + bounds.h > content_region.y + content_region.h + (float)offsety) {
    nk_window_set_scroll(widget->ctx, offsetx, (nk_uint)(bounds.y + bounds.h - content_region.y - content_region.h));
}
else if (bounds.y < content_region.y + (float)offsety) {
    nk_window_set_scroll(widget->ctx, offsetx, (nk_uint)(bounds.y - content_region.y));
}

This part makes it so that when you're moving down in the available options with the gamepad, the window will also scroll down to show the new items.

Since when you're scrolling the window with the mouse, you're not actually changing the selected item, so it forcably scrolls the window to show the active item. Not sure the best solution around this.

Unsure of the best solution around this. Maybe we have to check if they're using the mouse or something?

[StrikerTheHedgefox]

OK, thanks for pointing me to the code that handles this. I'll try to come up with an intuitive solution. Personally, I think making this function only trigger when actively moving between active items by pointing at the object, or using the navigation keys would be ideal.

[RobLoach]

Oh, that's a really good idea.

[StrikerTheHedgefox]

Working on it. I'll get a PR ready soon.

[StrikerTheHedgefox]

How's this look? https://shadowmavericks.com/files/ShareX/tfury_2025-12-18_02-18-26.mp4

[RobLoach]

OMG IT WORKS