No wrap around for Jump tables

[SergeTauger]

When I create a menu as shown in muiref/Scrollable jump table and start scrolling through the menu, if new U8X8_MSG_GPIO_MENU_NEXT event is received and cursor is at last line, cursor is placed at the first line of the menu (wrap-around happens). How can I disable wrap-around, i.e. make U8X8_MSG_GPIO_MENU_NEXT have no effect if cursor is at the last line and make U8X8_MSG_GPIO_MENU_PREV have no effect if cursor is at the first line?

[olikraus]

The simple answer is, use this code instead of mui_u8g2_u16_list_goto_w1_pi:

u8g2/sys/sdl/mui_issue_2597/main.c

Lines 51 to 104 in 2b75f93

	uint8_t my_u16_list_goto_w1_pi(mui_t *ui, uint8_t msg)
	{
	uint8_t arg = ui->arg; /* arg provided with MUI_XYA() */
	switch(msg)
	{
	case MUIF_MSG_EVENT_NEXT:
	if ( arg+1 == ui->form_scroll_visible )
	{
	if ( ui->form_scroll_visible + ui->form_scroll_top < ui->form_scroll_total )
	{
	ui->form_scroll_top++;
	return 1;
	}
	else
	{
	/* no wrap around to the top */
	return 1;
	/*
	ui->form_scroll_top = 0;
	*/
	}
	}
	break;
	case MUIF_MSG_EVENT_PREV:
	if ( arg == 0 ) /* only if we are at the topmost field */
	{
	if ( ui->form_scroll_top > 0 )
	{
	ui->form_scroll_top--;
	return 1;
	}
	else
	{
	/* no wrap around to the button */
	return 1;
	/*
	if ( ui->form_scroll_total > ui->form_scroll_visible )
	{
	ui->form_scroll_top = ui->form_scroll_total - ui->form_scroll_visible;
	}
	else
	{
	ui->form_scroll_top = 0;
	}
	*/
	}
	}
	break;
	default:
	return mui_u8g2_u16_list_goto_w1_pi(ui, msg);
	}
	return 0;
	}

The idea is to take an existing MUIF (here mui_u8g2_u16_list_goto_w1_pi) and extend / change the functionality. In this case we reimplement the PREV/NEXT events. I just tookover the original code and removed those parts, which are responsible for the wrap around.

Hope this helps...

[SergeTauger]

Thank you, that's exactly I was looking for. Also, nice to see a good SOLID example.