curses: honor menu_next_page/&c option settings

tty and X11 honor the menu_xxx options.  Qt currently doesn't
support menu manipulation by keyboard.  curses does support that
but was only handling the default menu keys.

diff --git a/doc/fixes37.0 b/doc/fixes37.0
index 76d9b7e2fd0624ce61cf29001eb6f70f82696da0..2503a1b224a2c0c8df4d5f222eae907dd1ea5e1f 100644 (file)
--- a/doc/fixes37.0
+++ b/doc/fixes37.0
@@ -378,6 +378,7 @@ spells that require a target spot rather than a direction (like skilled
 prevent wish request "death wand" from matching Death monster and producing a
        random wand instead of a wand of death
 grammar bit: "you hear a [AEF] note squeak in the distance" (should be "an")
+curses interface failed to honor menu_xxx option settings for menu interaction
 
 
 Fixes to 3.7.0-x Problems that Were Exposed Via git Repository

diff --git a/win/curses/cursdial.c b/win/curses/cursdial.c
index 67fae2c10be22d9bacf378f2d23030b4f725e2e6..999db17f4ac7a2785ff56bdef6646ef60dd9305e 100644 (file)
--- a/win/curses/cursdial.c
+++ b/win/curses/cursdial.c
@@ -1305,7 +1305,7 @@ menu_display_page(nhmenu *menu, WINDOW * win, int page_num, char *selectors)
 static int
 menu_get_selections(WINDOW * win, nhmenu *menu, int how)
 {
-    int curletter;
+    int curletter, menucmd;
     int count = -1;
     int count_letter = '\0';
     int curpage = !menu->bottom_heavy ? 1 : menu->num_pages;
@@ -1342,7 +1342,9 @@ menu_get_selections(WINDOW * win, nhmenu *menu, int how)
             break;
         case PICK_ANY:
             if (curletter <= 0 || curletter >= 256 || !selectors[curletter]) {
-                switch (curletter) {
+                menucmd = (curletter <= 0 || curletter >= 255) ? curletter
+                          : (int) (uchar) map_menu_cmd(curletter);
+                switch (menucmd) {
                 case MENU_SELECT_PAGE:
                     (void) menu_operation(win, menu, SELECT, curpage);
                     break;
@@ -1377,7 +1379,9 @@ menu_get_selections(WINDOW * win, nhmenu *menu, int how)
         }
 
         if (curletter <= 0 || curletter >= 256 || !selectors[curletter]) {
-            switch (curletter) {
+            menucmd = (curletter <= 0 || curletter >= 255) ? curletter
+                      : (int) (uchar) map_menu_cmd(curletter);
+            switch (menucmd) {
             case KEY_ESC:
                 num_selected = -1;
                 dismiss = TRUE;