After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 95747 - Should ignore EnterNotify events with the NotifyInferior detail set
Should ignore EnterNotify events with the NotifyInferior detail set
Status: RESOLVED FIXED
Product: metacity
Classification: Other
Component: general
unspecified
Other other
: Normal normal
: ---
Assigned To: Metacity maintainers list
Metacity maintainers list
Depends on:
Blocks:
 
 
Reported: 2002-10-14 18:35 UTC by Federico Mena Quintero
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.0

Attachments
Patch with trivial fix. (1.19 KB, patch)
2002-10-14 22:43 UTC, Federico Mena Quintero 		none Details | Review

Description Federico Mena Quintero 2002-10-14 18:35:34 UTC 
Metacity should not give the focus to a window when it gets an EnterNotify
event which has the detail field set to NotifyInferior.

This leads to a lot of flicker in the Evolution message composer when a
selection is active.  If the HTML widget has the focus and you move the
mouse over the "To:"/"CC:"/etc. fileds, then Metacity will re-set the focus
on the toplevel composer window, causing flicker.

Sawfish does ignore EnterNotify events for focusing when they have the
NotifyInferior detail set, btw.

(Metacity correctly ignores LeaveNotify events when they are NotifyInferior.)
Comment 1 Havoc Pennington 2002-10-14 19:38:15 UTC 

*** This bug has been marked as a duplicate of 90382 ***
Comment 2 Havoc Pennington 2002-10-14 22:22:33 UTC 
I'm on a lot of drugs; this isn't a dup.
Comment 3 Federico Mena Quintero 2002-10-14 22:43:02 UTC 
Created attachment 11539 [details] [review]
Patch with trivial fix.
Comment 4 Havoc Pennington 2002-10-15 00:06:51 UTC 
Thanks, feel free to commit.
Comment 5 Federico Mena Quintero 2002-10-15 14:46:07 UTC 
Cool, thanks.  It's in CVS now.