<div class="gmail_quote">2009/9/24 András Murányi <span dir="ltr">&lt;<a href="mailto:muranyia@gmail.com" target="_blank">muranyia@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">



<div><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div><div><div><blockquote type="cite">
<br>In the meantime, you could assign me small jobs on the tcl side, if
any. I could for example take a look if the popup menu can be built in
a way that behaves better with compiz..?<br></blockquote><div><br></div></div>

</div><div>Please do!  That would be great.  There are lots of TODOs in
the source code, plus some on the wiki.  Most are totally open for
anyone who want to take them on.  Just post here to express your
interest so that people don&#39;t step each others&#39; toes.</div>

</div><div><br></div></div></blockquote><div> </div></div></div>As for the menu it proves to be an unavoidable speciality of tk. I think it&#39;s a bug, so i have filed it.<br clear="all"></blockquote><div><br>Seems it&#39;s fixed in 8.5 now (see below)<br>

I noticed Hans had changed submenu delays to infinite - if it was to avoid compiz glitches it can be changed back now.<br></div></div>
<br>Andras<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">SourceForge.net</b> <span dir="ltr">&lt;<a href="mailto:noreply@sourceforge.net" target="_blank">noreply@sourceforge.net</a>&gt;</span><br>



Date: Tue, Dec 8, 2009 at 10:50 PM<br>Subject: [ tktoolkit-Bugs-2864685 ] Compiz animates each menu item as a new window<br>To: <a href="mailto:noreply@sourceforge.net" target="_blank">noreply@sourceforge.net</a><br><br>


<br>Bugs item #2864685, was opened at 2009-09-23 01:06<br>

Message generated for change (Settings changed) made by patthoyts<br>
You can respond by visiting:<br>
<a href="https://sourceforge.net/tracker/?func=detail&amp;atid=112997&amp;aid=2864685&amp;group_id=12997" target="_blank">https://sourceforge.net/tracker/?func=detail&amp;atid=112997&amp;aid=2864685&amp;group_id=12997</a><br>




<br>
Please note that this message will contain a full copy of the comment thread,<br>
including the initial issue submission, for this request,<br>
not just the latest update.<br>
Category: 12. Unix Menus<br>
Group: obsolete: 8.5.7<br>
&gt;Status: Closed<br>
&gt;Resolution: Fixed<br>
Priority: 9<br>
Private: No<br>
Submitted By: Andras Muranyi (muranyia)<br>
Assigned to: Donal K. Fellows (dkf)<br>
Summary: Compiz animates each menu item as a new window<br>
<br>
Initial Comment:<br>
Compiz somehow thinks that the menus of these apps are separate windows and applies window open and close animation to them. Thus each submenu &quot;pulses&quot; like ordinary window when opened and navigating a menu (specially one with zero delay) looks like a firework.<br>




<br>
----------------------------------------------------------------------<br>
<br>
&gt;Comment By: Pat Thoyts (patthoyts)<br>
Date: 2009-12-08 21:50<br>
<br>
Message:<br>
Applied this patch to HEAD and to 8.5 branch.<br>
A separate patch (possibly done as a TIP) will be done to make it possible<br>
to handle the combobox dropdowns, tooltips and other window types from<br>
script level.<br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Emiliano (egavilan)<br>
Date: 2009-12-07 19:27<br>
<br>
Message:<br>
Sorry, forgot to attach patch&#39;s URL:<br>
<a href="http://paste.tclers.tk/1887" target="_blank">http://paste.tclers.tk/1887</a><br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Emiliano (egavilan)<br>
Date: 2009-12-07 19:26<br>
<br>
Message:<br>
Updated patch with feedback and code from patthoyts.<br>
Menus show sensible behaviour in both normal and tornoff states now.<br>
Patch against HEAD.<br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Emiliano (egavilan)<br>
Date: 2009-12-07 15:01<br>
<br>
Message:<br>
You are right (I was reading an old version of the document).<br>
I tried setting _NET_WM_WINDOW_TYPE_MENU for torn-off menus and<br>
_NET_WM_WINDOW_TYPE_DROPDOWN_MENU for normal menus. The result is the<br>
same for normal (override-redirect) menus, but for torn-off menus the<br>
result is a window which doesn&#39;t have an icon in the applications bar<br>
but is not transient of the [winfo toplevel $menu] . This can resut in the<br>
menu going bottom of the window stack without the user noticing.<br>
<br>
I think we can do two things in this scenario:<br>
* For torn-off menus, we set _NET_WM_WINDOW_TYPE_NORMAL and it behaves<br>
like a normal (animated) window. This is the default now.<br>
* For torn-off menus, we set _NET_WM_WINDOW_TYPE_MENU and<br>
WM_TRANSIENT_FOR to make the menu transient of its toplevel, so it<br>
doesn&#39;t get lost behind other windows.<br>
<br>
I don&#39;t think Tk can distinguish between a &quot;normal&quot; menu and a &quot;popup&quot;<br>
menu, so _NET_WM_WINDOW_TYPE_POPUP_MENU is out of discussion.<br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Donal K. Fellows (dkf)<br>
Date: 2009-12-06 17:32<br>
<br>
Message:<br>
Note that these additional properties were not described in EWMH 1.3, but<br>
instead form part of EWMH 1.4.<br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Donal K. Fellows (dkf)<br>
Date: 2009-12-06 17:02<br>
<br>
Message:<br>
Actually, it wants to be one of:<br>
1. _NET_WM_WINDOW_TYPE_MENU - for torn off menus<br>
2. _NET_WM_WINDOW_TYPE_DROPDOWN_MENU - for &quot;real&quot; menus from the menubar<br>
3. _NET_WM_WINDOW_TYPE_POPUP_MENU - for menus that are neither of the<br>
other two (i.e. posted with tk_popup)<br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Emiliano (egavilan)<br>
Date: 2009-12-05 17:21<br>
<br>
Message:<br>
Apparently new window managers expects additional information to be set<br>
about what the function of the window is.<br>
More precisely, it expects the _NET_WM_WINDOW_TYPE property to be set to<br>
_NET_WM_WINDOW_TYPE_MENU for menu windows. See<br>
<a href="http://standards.freedesktop.org/wm-spec/1.3/ar01s05.html" target="_blank">http://standards.freedesktop.org/wm-spec/1.3/ar01s05.html</a> .<br>
A patch against HEAD (5 dec 2009) adds this information to menu windows on<br>
x11.<br>
As a result, the new property is set for menu windows as shown by xprop<br>
output:<br>
<br>
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_MENU<br>
<br>
For the patch, see <a href="http://paste.tclers.tk/1880" target="_blank">http://paste.tclers.tk/1880</a><br>
<br>
----------------------------------------------------------------------<br>
<br>
Comment By: Donal K. Fellows (dkf)<br>
Date: 2009-11-29 18:43<br>
<br>
Message:<br>
The menu code is very scattered over different files, but the key part<br>
appears to be in tkUnixWm.c in the function to create the actual X window<br>
for the menu (TkpMakeMenuWindow). If I&#39;m reading that function right, the<br>
override_redirect flag is not being set on the menu until after the window<br>
is created, and maybe compiz is not noticing that. Can&#39;t tell, as I&#39;m not a<br>
user of the right platform...<br>
<br>
----------------------------------------------------------------------<br>
<br>
You can respond by visiting:<br>
<a href="https://sourceforge.net/tracker/?func=detail&amp;atid=112997&amp;aid=2864685&amp;group_id=12997" target="_blank">https://sourceforge.net/tracker/?func=detail&amp;atid=112997&amp;aid=2864685&amp;group_id=12997</a><br>




</div><br><br clear="all"><br>