[PATCH 1/2] wpa_gui-qt4: add status fields to tray message

[Jouni Malinen]

On Fri, Sep 26, 2008 at 05:22:45AM +1000, Kel Modderman wrote:

> It is a visible hint to the user that the tray icon exists, and the program
> is still running. Rather than some bland text I thought maybe it was better
> just to display a status bubble.

OK. I'm a bit biased as far as unsolicited status messages popping up on
the screen are concerned since I think most of them are there just to
distract me from whatever I was doing.. ;-)

Since the behavior of the close button changed from the previous
version, users may be surprised of the program remaining running and as
such, I can see the desire to make it clear to the user what happened.
If there is CPU and GPU resources to waste, some kind of animation of
the window moving into the tray without any text could be ideal, but I
don't know whether there is any standard functionality in Qt/KDE for
such a thing. Barring that, having a minimal text notification sounds
reasonable.

> Okay, I understand how my idea was a little counter-intuitive, please consider
> the attached patch.

> Don't bother showing a status message when returning to the system tray,
> but still show a one time tray message to indicate to the user that the
> program is still running in the tray if they triggered the window manager
> close button.

Applied. I think this takes care of the original intent in a clearer and
less distracting way.

-- 
Jouni Malinen                                            PGP id EFC895FA