all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Robin Green <greenrd@greenrd.org>
To: 4122@emacsbugs.donarmstrong.com
Subject: bug#4122: Quick fix
Date: Mon, 31 Aug 2009 01:28:51 +0100	[thread overview]
Message-ID: <20090831012851.1ba99285@fedora> (raw)

[-- Attachment #1: Type: text/plain, Size: 494 bytes --]

The attached patch against Emacs 23.1.1 fixes this bug for me. The
patch can be used by emacs users and Linux distributors - who might
need to lower the required GTK version. However, the patch would
probably need to be modified before being checked in to emacs CVS,
because it specifies a dependency on a very recent version of GTK - a
more suitable patch for emacs CVS would use the C preprocessor to
perform the old behaviour for old GTK versions, and the new behaviour
for new GTK versions.

[-- Attachment #2: emacs-gtk.patch --]
[-- Type: text/x-patch, Size: 1515 bytes --]

--- configure.in.orig	2009-08-31 00:18:44.000000000 +0100
+++ configure.in	2009-08-31 00:18:47.000000000 +0100
@@ -1514,7 +1514,7 @@
 HAVE_GTK=no
 if test "${with_gtk}" = "yes" || test "$USE_X_TOOLKIT" = "maybe"; then
   GLIB_REQUIRED=2.6
-  GTK_REQUIRED=2.6
+  GTK_REQUIRED=2.17.9
   GTK_MODULES="gtk+-2.0 >= $GTK_REQUIRED glib-2.0 >= $GLIB_REQUIRED"
 
   dnl Checks for libraries.
--- src/xfns.c.orig	2009-08-31 00:03:02.000000000 +0100
+++ src/xfns.c	2009-08-31 00:21:39.000000000 +0100
@@ -468,8 +468,10 @@
       if (!FRAME_X_P (f) || FRAME_X_DISPLAY_INFO (f) != dpyinfo)
 	continue;
       x = f->output_data.x;
-      /* Match if the window is this frame's menubar.  */
 #ifdef USE_GTK
+      /* Match if the window is this frame's window (button presses are no longer
+         caught by the menubar) or a menu in the menubar.
+       */
       if (x->menubar_widget)
         {
           GtkWidget *gwdesc = xg_win_to_widget (dpyinfo->display, wdesc);
@@ -477,13 +479,14 @@
 
           BLOCK_INPUT;
           if (gwdesc != 0
-              && (gwdesc == x->menubar_widget
+              && (gwdesc == gtk_widget_get_parent (gtk_widget_get_parent (x->menubar_widget))
                   || gtk_widget_get_parent (gwdesc) == x->menubar_widget))
             found = 1;
           UNBLOCK_INPUT;
           if (found) return f;
         }
 #else
+      /* Match if the window is in this frame's menubar */
       if (x->menubar_widget
 	  && lw_window_is_in_menubar (wdesc, x->menubar_widget))
 	return f;

             reply	other threads:[~2009-08-31  0:28 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-31  0:28 Robin Green [this message]
2009-09-01 10:43 ` bug#4122: Quick fix Jan D.

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090831012851.1ba99285@fedora \
    --to=greenrd@greenrd.org \
    --cc=4122@emacsbugs.donarmstrong.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.