[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-windows
Subject: Re: KSystemTrayIcon on system without X11
From: Carlo <brandon.ml () gmail ! com>
Date: 2008-04-06 16:18:49
Message-ID: 3262b6180804060918p5dabb0bbjc062a8cfdaad0c65 () mail ! gmail ! com
[Download RAW message or body]
On Sun, Apr 6, 2008 at 5:16 PM, Christian Ehrlicher <Ch.Ehrlicher@gmx.de> wrote:
> Carlo schrieb:
>
>
>
> > On Sun, Apr 6, 2008 at 10:08 AM, Christian Ehrlicher
> > <Ch.Ehrlicher@gmx.de> wrote:
> >
> > > Carlo schrieb:
> > >
> > >
> > >
> > >
> > > > Hi,
> > > > right now clicking on a KSystemTrayIcon to show/hide the main window
> > > > works only on X11 because there is an ifdef with X11 specific code, so
> > > > i've made a patch to use setVisible on other platforms but i was
> > > > wondering if it's a problem to use setVisibile or the x11 code is just
> > > > to handle specific features like multiple desktops
> > > >
> > > >
> > > >
> > > Your implementation is wrong. I'm currently trying to figure out how to
> > > make KSystemTrayIcon::activateOrHide() behave correct on windows.
> > > KSystemTrayIcon::minimizeRestore() already works and I can check it in
> if
> > > you want.
> > >
> > >
> > > Christian
> > >
> > >
> > >
> > Why is wrong? seems to work for me
> >
> >
> There's a difference between 'working' and 'working like described'.
> See here:
>
> // if the window is not the active one, show it if needed, and activate it
> // (just like taskbar); otherwise hide it
> void KSystemTrayIcon::activateOrHide( QSystemTrayIcon::ActivationReason
> reasonCalled )
> You just make it visible or not.
>
> And minimizeRestore() also brings the window on top of the z-order, you
> patch doesn't do this.
>
>
> Christian
>
>
Ah yes of course it's just a simple hide/show and i tought that's
better than nothing for now, but if you are working to make it behave
like on X11 it's even better
Carlo
_______________________________________________
Kde-windows mailing list
Kde-windows@kde.org
https://mail.kde.org/mailman/listinfo/kde-windows
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic