Bug 168879 - kget cancels kde logout
Summary: kget cancels kde logout
Status: RESOLVED FIXED
Alias: None
Product: kget
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: KGet authors
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-10 23:58 UTC by S. Burmeister
Modified: 2008-08-12 20:31 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
quit kget if the close event comes from system shutdown (780 bytes, patch)
2008-08-11 20:07 UTC, Javier goday
Details
set mainwindow WA_DeleteOnClose = false (1.21 KB, patch)
2008-08-12 07:46 UTC, Javier goday
Details

Note You need to log in before you can comment on or make changes to this bug.
Description S. Burmeister 2008-08-10 23:58:59 UTC
Version:            (using KDE 4.1.0)
Installed from:    SuSE RPMs
OS:                Linux

If you have the kget window open and try to log out of kde, kget minimises to tray and the log out is canceled.

I guess there is some bug in the close event handling.

Expected behaviour, save d/l list and exit in order to let kde shut down.
Comment 1 Javier goday 2008-08-11 20:07:40 UTC
Created attachment 26788 [details]
quit kget if the close event comes from system shutdown

the problem is in the mainwindow->closeEvent, if the systemtray is enabled the
event is ignored. So we need to decide if we want to quit kget or only hide it
Comment 2 S. Burmeister 2008-08-11 20:17:05 UTC
> the problem is in the mainwindow->closeEvent, if the systemtray is enabled
> the event is ignored. So we need to decide if we want to quit kget or only
> hide it


Can't you just have a look at the other applications, that can be minimised to 
tray, yet do not interrupt logout? Like amarok, kopete or kgpg etc.
Comment 3 Javier goday 2008-08-12 07:46:38 UTC
Created attachment 26796 [details]
set mainwindow WA_DeleteOnClose = false

Set mainwindow DeleteOnClose attribute = false, so don't need to reimplement
closeEvent method
Comment 4 Javier goday 2008-08-12 20:31:47 UTC
SVN commit 845969 by jgoday:

Don't ignore the close event when shutdown
BUG:168879


 M  +15 -8     mainwindow.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=845969