home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / old / 5.6.029 < prev    next >
Encoding:
Internet Message Format  |  2000-03-24  |  4.5 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 5.6.029
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. ------------
  6.  
  7. Patch 5.6.029 
  8. Problem:    GTK GUI: Shortcut keys cannot be used for a dialog. (Johannes
  9.             Zellner)
  10. Solution:   Add support for shortcut keys. (Marcin Dalecki)
  11. Files:      src/gui_gtk.c
  12.  
  13.  
  14. *** ../vim-5.6.28/src/gui_gtk.c    Fri Jan 21 11:10:27 2000
  15. --- src/gui_gtk.c    Sat Mar 25 21:08:19 2000
  16. ***************
  17. *** 947,952 ****
  18. --- 947,953 ----
  19.   {
  20.       char_u *names;
  21.       char_u *p;
  22. +     int i;
  23.       int butcount;
  24.       int dialog_status = -1;
  25.   
  26. ***************
  27. *** 959,964 ****
  28. --- 960,966 ----
  29.       GtkWidget *action_area;
  30.       GtkWidget *sub_area;
  31.       GtkWidget *separator;
  32. +     GtkAccelGroup *accel_group;
  33.   
  34.       GdkPixmap *icon = NULL;
  35.       GdkBitmap *mask = NULL;
  36. ***************
  37. *** 1057,1086 ****
  38.       gtk_box_pack_start(GTK_BOX(action_area), sub_area, FALSE, TRUE, 0);
  39.       gtk_widget_show(sub_area);
  40.   
  41. -     /* make a copy, so that we can insert NULs */
  42. -     names = vim_strsave(buttons);
  43. -     if (names == NULL)
  44. -     return -1;
  45. - 
  46.       /*
  47.        * Create the buttons.
  48.        */
  49.       p = names;
  50.       for (butcount = 0; butcount < MAXBUT; ++butcount) {
  51.       char_u *next;
  52.   
  53.       for (next = p; *next; ++next) {
  54. -         if (*next == DLG_HOTKEY_CHAR)
  55. -         mch_memmove(next, next + 1, STRLEN(next));
  56.           if (*next == DLG_BUTTON_SEP) {
  57.           *next++ = NUL;
  58.           break;
  59.           }
  60.       }
  61.   
  62. !     button[butcount] = gtk_button_new_with_label((const gchar *)p);
  63.       GTK_WIDGET_SET_FLAGS(button[butcount], GTK_CAN_DEFAULT);
  64. !     gtk_widget_show(button[butcount]);
  65.   
  66.       data[butcount].status = &dialog_status;
  67.       data[butcount].index = butcount;
  68. --- 1059,1135 ----
  69.       gtk_box_pack_start(GTK_BOX(action_area), sub_area, FALSE, TRUE, 0);
  70.       gtk_widget_show(sub_area);
  71.   
  72.       /*
  73.        * Create the buttons.
  74.        */
  75. + 
  76. +     /*
  77. +      * Translate the Vim accelerator character into an underscore for GTK+.
  78. +      * Double underscores to keep them in the label.
  79. +      */
  80. +     /* count the number of underscores */
  81. +     i = 1;
  82. +     for (p = buttons; *p; ++p)
  83. +     if (*p == '_')
  84. +         ++i;
  85. + 
  86. +     /* make a copy of "buttons" with the translated characters */
  87. +     names = alloc(STRLEN(buttons) + i);
  88. +     if (names == NULL)
  89. +     return -1;
  90. + 
  91. +     p = names;
  92. +     for (i = 0; buttons[i]; ++i)
  93. +     {
  94. +     if (buttons[i] == DLG_HOTKEY_CHAR)
  95. +         *p++ = '_';
  96. +     else
  97. +     {
  98. +         if (buttons[i] == '_')
  99. +         *p++ = '_';
  100. +         *p++ = buttons[i];
  101. +     }
  102. +     }
  103. +     *p = NUL;
  104. + 
  105. +     /* Attach the new accelerator group to the window. */
  106. +     accel_group = gtk_accel_group_new();
  107. +     gtk_accel_group_attach(accel_group, GTK_OBJECT(dialog));
  108. + 
  109.       p = names;
  110.       for (butcount = 0; butcount < MAXBUT; ++butcount) {
  111.       char_u *next;
  112. +     GtkWidget *label;
  113. +     guint accel_key;
  114.   
  115. +     /* Chunk out this single button. */
  116.       for (next = p; *next; ++next) {
  117.           if (*next == DLG_BUTTON_SEP) {
  118.           *next++ = NUL;
  119.           break;
  120.           }
  121.       }
  122.   
  123. !     button[butcount] = gtk_button_new();
  124.       GTK_WIDGET_SET_FLAGS(button[butcount], GTK_CAN_DEFAULT);
  125. ! 
  126. !     label = gtk_accel_label_new("");
  127. !         gtk_accel_label_set_accel_widget(GTK_ACCEL_LABEL(label), dialog);
  128. ! 
  129. !     accel_key = gtk_label_parse_uline(GTK_LABEL(label), p);
  130. ! # ifdef GTK_USE_ACCEL
  131. !     /* Don't add accelator if 'winaltkeys' is "no". */
  132. !     if (accel_key != GDK_VoidSymbol) {
  133. !         gtk_widget_add_accelerator(button[butcount],
  134. !             "clicked",
  135. !             accel_group,
  136. !             accel_key, 0,
  137. !             0);
  138. !     }
  139. ! # endif
  140. ! 
  141. !     gtk_container_add(GTK_CONTAINER(button[butcount]), label);
  142. !     gtk_widget_show_all(button[butcount]);
  143.   
  144.       data[butcount].status = &dialog_status;
  145.       data[butcount].index = butcount;
  146. ***************
  147. *** 1100,1105 ****
  148. --- 1149,1155 ----
  149.       }
  150.       p = next;
  151.       }
  152. + 
  153.       vim_free(names);
  154.   
  155.       --def_but;        /* 1 is first button */
  156. ***************
  157. *** 1124,1129 ****
  158. --- 1174,1183 ----
  159.   
  160.       if (dialog_status < 0)
  161.       dialog_status = 0;
  162. + 
  163. +     /* let the garbage collector know that we don't need it anylonger */
  164. +     gtk_accel_group_unref(accel_group);
  165. + 
  166.       return dialog_status;
  167.   }
  168.   
  169. *** ../vim-5.6.28/src/version.c    Sat Mar 25 20:39:12 2000
  170. --- src/version.c    Sat Mar 25 21:19:02 2000
  171. ***************
  172. *** 420,421 ****
  173. --- 420,423 ----
  174.   {   /* Add new patch number below this line */
  175. + /**/
  176. +     29,
  177.   /**/
  178.  
  179. -- 
  180. TALL KNIGHT OF NI: Ni!
  181.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  182.  
  183. /-/-- Bram Moolenaar --- Bram@moolenaar.net --- http://www.moolenaar.net --\-\
  184. \-\-- Vim: http://www.vim.org ---- ICCF Holland: http://www.vim.org/iccf --/-/
  185.