6.2.293

   1 To: vim-dev@vim.org
   2 Subject: Patch 6.2.293
   3 Fcc: outbox
   4 From: Bram Moolenaar <Bram@moolenaar.net>
   5 Mime-Version: 1.0
   6 Content-Type: text/plain; charset=ISO-8859-1
   7 Content-Transfer-Encoding: 8bit
   8 ------------
   9
  10 Patch 6.2.293 (after 6.2.255)
  11 Problem:    GTK: A new item in a menu is put before the tearoff item.
  12 Solution:   Do increment the menu item index for non-popup menu items.
  13 Files:      src/gui_gtk.c
  14
  15
  16 *** ../vim-6.2.292/src/gui_gtk.c        Sun Feb 15 13:42:25 2004
  17 --- src/gui_gtk.c       Thu Feb 26 17:47:49 2004
  18 ***************
  19 *** 575,582 ****
  20       parent_widget = (parent != NULL) ? parent->submenu_id : gui.menubar;
  21       menu_item_new(menu, parent_widget);
  22
  23 !     if (parent != NULL)
  24 !       /* since the tearoff should always appear first, increment idx */
  25         ++idx;
  26
  27       gtk_menu_shell_insert(GTK_MENU_SHELL(parent_widget), menu->id, idx);
  28 --- 575,582 ----
  29       parent_widget = (parent != NULL) ? parent->submenu_id : gui.menubar;
  30       menu_item_new(menu, parent_widget);
  31
  32 !     /* since the tearoff should always appear first, increment idx */
  33 !     if (parent != NULL && !menu_is_popup(parent->name))
  34         ++idx;
  35
  36       gtk_menu_shell_insert(GTK_MENU_SHELL(parent_widget), menu->id, idx);
  37 ***************
  38 *** 804,812 ****
  39         if (parent->submenu_id == NULL)
  40             return;
  41
  42 !       /* make place for the possible tearoff handle item */
  43 !       /* ++idx; Don't understand why this was here; makes adding an item to
  44 !        * the popup menu appear after instead of before an existing item. */
  45
  46         if (menu_is_separator(menu->name))
  47         {
  48 --- 804,813 ----
  49         if (parent->submenu_id == NULL)
  50             return;
  51
  52 !       /* Make place for the possible tearoff handle item.  Not in the popup
  53 !        * menu, it doesn't have a tearoff item. */
  54 !       if (parent != NULL && !menu_is_popup(parent->name))
  55 !           ++idx;
  56
  57         if (menu_is_separator(menu->name))
  58         {
  59 *** ../vim-6.2.292/src/version.c        Thu Feb 26 15:42:54 2004
  60 --- src/version.c       Thu Feb 26 18:15:20 2004
  61 ***************
  62 *** 639,640 ****
  63 --- 639,642 ----
  64   {   /* Add new patch number below this line */
  65 + /**/
  66 +     293,
  67   /**/
  68
  69 --
  70        Bravely bold Sir Robin, rode forth from Camelot,
  71        He was not afraid to die, Oh Brave Sir Robin,
  72        He was not at all afraid to be killed in nasty ways
  73        Brave, brave, brave, brave Sir Robin.
  74                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  75
  76  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  77 ///        Sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  78 \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  79  \\\  Buy at Amazon and help AIDS victims -- http://ICCF.nl/click1.html ///