[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: XBar development , asking for help when restoring top menu
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2010-08-24 17:48:54
Message-ID: 201008241048.55056.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Saturday, August 21, 2010, Aaron Lewis wrote:
> 	I'm currently working on a project, plasma-globalmenu , i tried source
> code from kde web svn interface , and found it buggy.

great that you are working on this.

Aurelien Gateau is working on a dbusmenu based global menubar as well; it has 
the advantage of allowing the client (in this case plasma) do all the 
painting, but the disadvantage of only working with apps that are patched to 
use dbusmenu.

i wonder if it might be possible to merge the two efforts at some point with 
the xbar method used as a fallback if dbusmenu isn't available, etc? hm...

anyways ...

> void GlobalMenu::activeWindowChanged(WId wid)
> {
> ..
>  	// Thus , when window is not active , remove menu , fall back to
> default XBar menu
> 	if(KWindowSystem::activeWindow() != WId) {

is this the actual, literal code you have in your project? because i can't see 
how this will even compile, as WId is a type not a value :) in any case, if 
you meant wid, then the if statement will never be true since 
activeWindowChanged is emitted once the new window is active. meaning that the 
WId passed in to your slot is KWindowSystem::activeWindow().

it should be comparing KWindowSystem::activeWindow() with the WId of the 
window that registered the current global menu.

oh, and plasma devel related discussion can be found on plasma-devel@kde.org 
if you are interested :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks

["signature.asc" (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread]