]> www.pilppa.org Git - linux-2.6-omap-h63xx.git/commitdiff
kconfig: Fix indention when using menuconfig in text-onle consoles
authorSam Ravnborg <sam@mars.ravnborg.org>
Sun, 20 Nov 2005 22:03:49 +0000 (23:03 +0100)
committerSam Ravnborg <sam@mars.ravnborg.org>
Sun, 20 Nov 2005 22:03:49 +0000 (23:03 +0100)
When using menuconfig in a text-only console (no X started)
the indention was often two spaces wrong. This proved to be a ncurses
issue which are worked around by calling wrefresh more often.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
scripts/lxdialog/menubox.c

index 89fcf418a07a4262f7530f07ab46bae3fe957bd0..461ee08edc05df95ea8155a3a1878846dfe27a14 100644 (file)
@@ -94,9 +94,9 @@ static void print_item(WINDOW * win, const char *item, int choice,
        }
        if (selected) {
                wmove(win, choice, ITEM_IDENT + 1);
        }
        if (selected) {
                wmove(win, choice, ITEM_IDENT + 1);
-               wrefresh(win);
        }
        free(menu_item);
        }
        free(menu_item);
+       wrefresh(win);
 }
 
 /*
 }
 
 /*
@@ -125,6 +125,7 @@ static void print_arrows(WINDOW * win, int item_no, int scroll, int y, int x,
 
        y = y + height + 1;
        wmove(win, y, x);
 
        y = y + height + 1;
        wmove(win, y, x);
+       wrefresh(win);
 
        if ((height < item_no) && (scroll + height < item_no)) {
                wattrset(win, darrow_attr);
 
        if ((height < item_no) && (scroll + height < item_no)) {
                wattrset(win, darrow_attr);
@@ -139,6 +140,7 @@ static void print_arrows(WINDOW * win, int item_no, int scroll, int y, int x,
        }
 
        wmove(win, cur_y, cur_x);
        }
 
        wmove(win, cur_y, cur_x);
+       wrefresh(win);
 }
 
 /*
 }
 
 /*
@@ -157,6 +159,17 @@ static void print_buttons(WINDOW * win, int height, int width, int selected)
        wrefresh(win);
 }
 
        wrefresh(win);
 }
 
+/* scroll up n lines (n may be negative) */
+static void do_scroll(WINDOW *win, int *scroll, int n)
+{
+       /* Scroll menu up */
+       scrollok(win, TRUE);
+       wscrl(win, n);
+       scrollok(win, FALSE);
+       *scroll = *scroll + n;
+       wrefresh(win);
+}
+
 /*
  * Display a menu for choosing among a number of options
  */
 /*
  * Display a menu for choosing among a number of options
  */
@@ -286,11 +299,7 @@ int dialog_menu(const char *title, const char *prompt, int height, int width,
                        if (key == KEY_UP || key == '-') {
                                if (choice < 2 && scroll) {
                                        /* Scroll menu down */
                        if (key == KEY_UP || key == '-') {
                                if (choice < 2 && scroll) {
                                        /* Scroll menu down */
-                                       scrollok(menu, TRUE);
-                                       wscrl(menu, -1);
-                                       scrollok(menu, FALSE);
-
-                                       scroll--;
+                                       do_scroll(menu, &scroll, -1);
 
                                        print_item(menu, items[scroll * 2 + 1], 0, FALSE,
                                                   (items[scroll * 2][0] != ':'));
 
                                        print_item(menu, items[scroll * 2 + 1], 0, FALSE,
                                                   (items[scroll * 2][0] != ':'));
@@ -306,11 +315,7 @@ int dialog_menu(const char *title, const char *prompt, int height, int width,
                                if ((choice > max_choice - 3) &&
                                    (scroll + max_choice < item_no)) {
                                        /* Scroll menu up */
                                if ((choice > max_choice - 3) &&
                                    (scroll + max_choice < item_no)) {
                                        /* Scroll menu up */
-                                       scrollok(menu, TRUE);
-                                       wscrl(menu, 1);
-                                       scrollok(menu, FALSE);
-
-                                       scroll++;
+                                       do_scroll(menu, &scroll, 1);
 
                                        print_item(menu, items[(scroll + max_choice - 1) * 2 + 1],
                                                   max_choice - 1, FALSE,
 
                                        print_item(menu, items[(scroll + max_choice - 1) * 2 + 1],
                                                   max_choice - 1, FALSE,
@@ -322,8 +327,7 @@ int dialog_menu(const char *title, const char *prompt, int height, int width,
                                scrollok(menu, TRUE);
                                for (i = 0; (i < max_choice); i++) {
                                        if (scroll > 0) {
                                scrollok(menu, TRUE);
                                for (i = 0; (i < max_choice); i++) {
                                        if (scroll > 0) {
-                                               wscrl(menu, -1);
-                                               scroll--;
+                                               do_scroll(menu, &scroll, -1);
                                                print_item(menu, items[scroll * 2 + 1], 0, FALSE,
                                                           (items[scroll * 2][0] != ':'));
                                        } else {
                                                print_item(menu, items[scroll * 2 + 1], 0, FALSE,
                                                           (items[scroll * 2][0] != ':'));
                                        } else {
@@ -331,15 +335,11 @@ int dialog_menu(const char *title, const char *prompt, int height, int width,
                                                        choice--;
                                        }
                                }
                                                        choice--;
                                        }
                                }
-                               scrollok(menu, FALSE);
 
                        } else if (key == KEY_NPAGE) {
                                for (i = 0; (i < max_choice); i++) {
                                        if (scroll + max_choice < item_no) {
 
                        } else if (key == KEY_NPAGE) {
                                for (i = 0; (i < max_choice); i++) {
                                        if (scroll + max_choice < item_no) {
-                                               scrollok(menu, TRUE);
-                                               wscrl(menu, 1);
-                                               scrollok(menu, FALSE);
-                                               scroll++;
+                                               do_scroll(menu, &scroll, 1);
                                                print_item(menu, items[(scroll + max_choice - 1) * 2 + 1],
                                                           max_choice - 1, FALSE,
                                                           (items [(scroll + max_choice - 1) * 2][0] != ':'));
                                                print_item(menu, items[(scroll + max_choice - 1) * 2 + 1],
                                                           max_choice - 1, FALSE,
                                                           (items [(scroll + max_choice - 1) * 2][0] != ':'));
@@ -348,7 +348,6 @@ int dialog_menu(const char *title, const char *prompt, int height, int width,
                                                        choice++;
                                        }
                                }
                                                        choice++;
                                        }
                                }
-
                        } else
                                choice = i;
 
                        } else
                                choice = i;