Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[menu] Prevent separators with shortcut keys from being selected
Nothing currently prevents a menu separator from being assigned a
shortcut key, and then from being selected using that shortcut key.
This produces an inconsistency in the user interface, since separators
cannot be selected by other means of menu navigation (arrow keys, page
up/down, etc).

It would be trivial to prevent separators from being assigned shortcut
keys, but this would eliminate one potentially useful use case: having
a large menu and using shortcut keys to jump to a section within the
menu.

Fix by treating a shortcut key on a separator as equivalent to "select
the separator, then press the down arrow key".  This has the effect of
moving to the first non-separator menu item following the specified
separator, which is probably the most intuitive behaviour.  (The
existing logic for moving the selection already handles the various
nasty corner cases such as a menu ending with one or more separators.)

Reported-by: Ján ONDREJ (SAL) <ondrejj@salstar.sk>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
  • Loading branch information
mcb30 committed Mar 6, 2013
1 parent b8cbdbb commit c080251
Showing 1 changed file with 12 additions and 10 deletions.
22 changes: 12 additions & 10 deletions src/hci/tui/menu_ui.c
Expand Up @@ -247,13 +247,17 @@ static int menu_loop ( struct menu_ui *ui, struct menu_item **selected ) {
i = 0;
list_for_each_entry ( item, &ui->menu->items,
list ) {
if ( item->shortcut &&
( item->shortcut == key ) ) {
ui->selected = i;
if ( ! ( item->shortcut &&
( item->shortcut == key ) ) ) {
i++;
continue;
}
ui->selected = i;
if ( item->label ) {
chosen = 1;
break;
} else {
move = +1;
}
i++;
}
break;
}
Expand Down Expand Up @@ -284,12 +288,10 @@ static int menu_loop ( struct menu_ui *ui, struct menu_item **selected ) {
draw_menu_item ( ui, ui->selected );
}

/* Refuse to choose unlabelled items (i.e. separators) */
item = menu_item ( ui->menu, ui->selected );
if ( ! item->label )
chosen = 0;

/* Record selection */
item = menu_item ( ui->menu, ui->selected );
assert ( item != NULL );
assert ( item->label != NULL );
*selected = item;

} while ( ( rc == 0 ) && ! chosen );
Expand Down

0 comments on commit c080251

Please sign in to comment.