Review Request 126917: Fix check whether window is closed by session management
Wolfgang Bauer
wbauer at tmo.at
Thu Jan 28 20:05:10 GMT 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126917/
-----------------------------------------------------------
(Updated Jan. 28, 2016, 8:05 p.m.)
Status
------
This change has been marked as submitted.
Review request for Dolphin.
Changes
-------
Submitted with commit bf1dbee1d6a709983d9f5034c61e2f005fdc6aee by Wolfgang Bauer to branch Applications/15.12.
Bugs: 353548
https://bugs.kde.org/show_bug.cgi?id=353548
Repository: dolphin
Description
-------
qApp->isSessionRestored() specifies whether the application has been *restored* by session management. This is obviously wrong here. Use qApp->isSavingSession() instead.
Diffs
-----
src/dolphinmainwindow.cpp 81c55e8
Diff: https://git.reviewboard.kde.org/r/126917/diff/
Testing
-------
Open multiple tabs and logout. Dolphin doesn't ask for confirmation any more.
It still asks when closing the window manually.
Thanks,
Wolfgang Bauer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20160128/d1026e07/attachment.htm>
More information about the kfm-devel
mailing list