From: Klaus K. <lx...@kn...> - 2010-01-30 14:28:50
|
Hi List, I have been hunting for a bug for quite some time which has the following symptom: "When clicking a window name in the taskbar of lxpanel, the window gets the focus, but is not [always] raised to the top, and no desktop switch is performed in the case that the window belongs to a different desktop other than the current." <- lxpanel 0.5 observation For a while, I have blamed compiz-fusion, because openbox-lxde at least puts the window to front. Now I did some more research and found that 1. lxpanel 0.3.8.1 works fine, and also switches desktops, rotating the compiz desktop cube whenever necessary. 2. lxpanel 0.5 seems to be confused about some GDK-related screen parameters, giving the error message lxpanel : X error: BadMatch (Invalid Parameter Attributes) whenever a window in the taskbar is clicked, independent of the window manager. lxpanel 0.3.8.1 did not display this error. This brings me to the conclusion that window raising and focusing is done in lxpanel 0.5 in a different way, that does NOT work correctly, opposed to lxpanel 0.3 where it was still OK. Sorry this is still an unsharp error report, but I currently don't have anything better. Temporary solution (for me): reverting back to lxpanel 0.3. Unfortunately, this means that I cannot use any of the new 0.5 plugins. This topic has been discusseded briefly here, too: http://sourceforge.net/tracker/index.php?func=detail&aid=2834685&group_id=180858&atid=894869 but no solution or point to a place in the source where to fix it yet. Unless someone has a very quick fix for window raising, we are going back to lxpanel 0.3 in KNOPPIX until this is resolved. No sweat. Another pending question: How can the user activate programs running in the panel dock (such as nm-applet) using the keyboard? This is vitally important for accessibility. Also, being able to activate panel program icons would be nice-to-have, but not as important as reaching the docked applications. The lxpanelctl menu hack works fine for the menu, but does not solve reachabilit of other lxpanel items. Regards -Klaus |
From: Marty J. <mar...@co...> - 2010-01-30 20:36:35
|
I'll have a fix checked in shortly. I want to do just a little more testing. On 01/30/2010 09:12 AM, Klaus Knopper wrote: > Hi List, > > I have been hunting for a bug for quite some time which has the > following symptom: > > "When clicking a window name in the taskbar of lxpanel, the window gets > the focus, but is not [always] raised to the top, and no desktop switch is > performed in the case that the window belongs to a different desktop > other than the current." <- lxpanel 0.5 observation > > For a while, I have blamed compiz-fusion, because openbox-lxde at least > puts the window to front. Now I did some more research and found that > > 1. lxpanel 0.3.8.1 works fine, and also switches desktops, rotating the > compiz desktop cube whenever necessary. > > 2. lxpanel 0.5 seems to be confused about some GDK-related screen > parameters, giving the error message > > lxpanel : X error: BadMatch (Invalid Parameter Attributes) > > whenever a window in the taskbar is clicked, independent of the window > manager. lxpanel 0.3.8.1 did not display this error. > > This brings me to the conclusion that window raising and focusing is > done in lxpanel 0.5 in a different way, that does NOT work correctly, > opposed to lxpanel 0.3 where it was still OK. Sorry this is still an > unsharp error report, but I currently don't have anything better. > > Temporary solution (for me): reverting back to lxpanel 0.3. > Unfortunately, this means that I cannot use any of the new 0.5 plugins. > > This topic has been discusseded briefly here, too: > http://sourceforge.net/tracker/index.php?func=detail&aid=2834685&group_id=180858&atid=894869 > but no solution or point to a place in the source where to fix it yet. > > Unless someone has a very quick fix for window raising, we are going > back to lxpanel 0.3 in KNOPPIX until this is resolved. No sweat. > > Another pending question: How can the user activate programs running in > the panel dock (such as nm-applet) using the keyboard? This is vitally > important for accessibility. Also, being able to activate panel program > icons would be nice-to-have, but not as important as reaching the docked > applications. The lxpanelctl menu hack works fine for the menu, but does > not solve reachabilit of other lxpanel items. > > Regards > -Klaus > > ------------------------------------------------------------------------------ > The Planet: dedicated and managed hosting, cloud storage, colocation > Stay online with enterprise data centers and the best network in the business > Choose flexible plans and management services without long-term contracts > Personal 24x7 support from experience hosting pros just a phone call away. > http://p.sf.net/sfu/theplanet-com > _______________________________________________ > Lxde-list mailing list > Lxd...@li... > https://lists.sourceforge.net/lists/listinfo/lxde-list > |
From: Marty J. <mar...@co...> - 2010-01-30 21:43:42
|
The fix is checked in as r2314. I would encourage those who are able to please test. There are two interrelated problems. One was introduced in PCMan's r1951 changes where by passing the wrong parameter he broke the evaluation of whether the window manager supports NET_WM_ACTIVE_WINDOW, which is normally the case in a modern window manager. Thus the taskbar was always going through the fallback code that uses XMapRaised and XSetInputFocus. The second is, there is a well known race condition in XSetInputFocus that causes the BadMatch error, for which I have also changed the workaround to a more reliable one. I would encourage everyone to pay attention to warnings. If you were to build 0.5.4.1 you would see taskbar.c:1783: warning: passing argument 1 of ‘gdk_x11_screen_supports_net_wm_hint’ from incompatible pointer type which could be an indication that you have introduced a bug. This is why some of the professional organizations I have worked for do not allow code to be released with any warnings. On 01/30/2010 03:36 PM, Marty Jack wrote: > I'll have a fix checked in shortly. I want to do just a little more testing. > > On 01/30/2010 09:12 AM, Klaus Knopper wrote: >> Hi List, >> >> I have been hunting for a bug for quite some time which has the >> following symptom: >> >> "When clicking a window name in the taskbar of lxpanel, the window gets >> the focus, but is not [always] raised to the top, and no desktop switch is >> performed in the case that the window belongs to a different desktop >> other than the current." <- lxpanel 0.5 observation >> >> For a while, I have blamed compiz-fusion, because openbox-lxde at least >> puts the window to front. Now I did some more research and found that >> >> 1. lxpanel 0.3.8.1 works fine, and also switches desktops, rotating the >> compiz desktop cube whenever necessary. >> >> 2. lxpanel 0.5 seems to be confused about some GDK-related screen >> parameters, giving the error message >> >> lxpanel : X error: BadMatch (Invalid Parameter Attributes) >> >> whenever a window in the taskbar is clicked, independent of the window >> manager. lxpanel 0.3.8.1 did not display this error. >> >> This brings me to the conclusion that window raising and focusing is >> done in lxpanel 0.5 in a different way, that does NOT work correctly, >> opposed to lxpanel 0.3 where it was still OK. Sorry this is still an >> unsharp error report, but I currently don't have anything better. >> >> Temporary solution (for me): reverting back to lxpanel 0.3. >> Unfortunately, this means that I cannot use any of the new 0.5 plugins. >> >> This topic has been discusseded briefly here, too: >> http://sourceforge.net/tracker/index.php?func=detail&aid=2834685&group_id=180858&atid=894869 >> but no solution or point to a place in the source where to fix it yet. >> >> Unless someone has a very quick fix for window raising, we are going >> back to lxpanel 0.3 in KNOPPIX until this is resolved. No sweat. >> >> Another pending question: How can the user activate programs running in >> the panel dock (such as nm-applet) using the keyboard? This is vitally >> important for accessibility. Also, being able to activate panel program >> icons would be nice-to-have, but not as important as reaching the docked >> applications. The lxpanelctl menu hack works fine for the menu, but does >> not solve reachabilit of other lxpanel items. >> >> Regards >> -Klaus >> >> ------------------------------------------------------------------------------ >> The Planet: dedicated and managed hosting, cloud storage, colocation >> Stay online with enterprise data centers and the best network in the business >> Choose flexible plans and management services without long-term contracts >> Personal 24x7 support from experience hosting pros just a phone call away. >> http://p.sf.net/sfu/theplanet-com >> _______________________________________________ >> Lxde-list mailing list >> Lxd...@li... >> https://lists.sourceforge.net/lists/listinfo/lxde-list >> > > ------------------------------------------------------------------------------ > The Planet: dedicated and managed hosting, cloud storage, colocation > Stay online with enterprise data centers and the best network in the business > Choose flexible plans and management services without long-term contracts > Personal 24x7 support from experience hosting pros just a phone call away. > http://p.sf.net/sfu/theplanet-com > _______________________________________________ > Lxde-list mailing list > Lxd...@li... > https://lists.sourceforge.net/lists/listinfo/lxde-list > |
From: Klaus K. <lx...@kn...> - 2010-01-31 00:10:13
|
Hello Marty & list, On Sat, Jan 30, 2010 at 04:43:34PM -0500, Marty Jack wrote: > The fix is checked in as r2314. I would encourage those who are able to please test. After applying your r2314 patch to the current Debian/unstable package xpanel version 0.5.4.1-1, I can confirm that window raising and desktop switching in compiz works again, and the debug message concerning missing gdk_x11_screen_supports_net_wm_hint is gone, as expected. Thanks a lot for your quick fix just in time for a new Knoppix (maintenance) release. :-) Regards -Klaus |