unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Jan D." <jan.h.d@swipnet.se>
Cc: Chong Yidong <cyd@stupidchicken.com>,
	Emacs Devel <emacs-devel@gnu.org>, "Kim F. Storm" <storm@cua.dk>
Subject: Re: Tool-bar buttons unresponsive with tooltips on
Date: Mon, 24 Oct 2005 07:02:07 +0200	[thread overview]
Message-ID: <DF6E8601-086D-4224-9247-3255361E7A9F@swipnet.se> (raw)
In-Reply-To: <uek6boq56.fsf@jasonrumney.net>


23 okt 2005 kl. 23.54 skrev Jason Rumney:

> "Jan D." <jan.h.d@swipnet.se> writes:
>
>
>> I've checked in this change.  I hope it doesn't ruin the intent of
>> calling remember_mouse_glyph in the first place.
>>
>
> The if statement is clearly trying to avoid calling  
> note_mouse_highlight
> for every pixel of mouse movement, since that was the only substantial
> code inside the if statement originally. Moving note_mouse_highlight
> outside the if block is certainly ruining the intent of that code.
>
> I don't understand what note_mouse_highlight has to do with being able
> to click the mouse on a toolbar button. I think if you can find the
> reason why note_mouse_highlight affects that, you will be led to the
> real bug.
>

I think the intent was to avoid doing frame->mouse_moved = 1, and  
note_mouse_highlight just happened to be there also.

The reason is that showing the tip window generates an EnterNotify  
event that clears dpyinfo->mouse_face_window (sets to Qnil).   
note_mouse_highlight calls note_tool_bar_highlight and that function  
sets mouse_face_window again to the tool bar window.  Later when  
checking if the click is in the tool bar, it is only recognised as a  
tool bar click if mouse_face_window is eq to the tool bar window.

Now, window managers are free to generate EnterNotify whenever they  
want and it seems that KDE generates one more than it used to, and  
also more than other window managers.  This is not a bug however.

     Jan D.

  reply	other threads:[~2005-10-24  5:02 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-19  8:39 [Chong Yidong] Re: Tool-bar buttons unresponsive with tooltips on David PONCE
2005-10-19 17:31 ` Jan D.
2005-10-19 17:57   ` Jan D.
2005-10-19 22:49     ` Chong Yidong
2005-10-20  9:06       ` Eli Zaretskii
2005-10-20 13:45       ` Kim F. Storm
2005-10-20 14:46         ` Chong Yidong
2005-10-23 21:14           ` Jan D.
2005-10-23 21:54             ` Jason Rumney
2005-10-24  5:02               ` Jan D. [this message]
2005-10-24  7:38                 ` Jason Rumney
2005-10-25 17:04                   ` Jan D.
2005-10-24  9:17             ` Kim F. Storm
2005-10-24 12:46               ` Chong Yidong
2005-10-24 21:44                 ` Kim F. Storm
  -- strict thread matches above, loose matches on Subject: below --
2005-10-14 20:27 Chong Yidong
2005-10-15 14:15 ` Chong Yidong

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=DF6E8601-086D-4224-9247-3255361E7A9F@swipnet.se \
    --to=jan.h.d@swipnet.se \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@gnu.org \
    --cc=storm@cua.dk \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).