Review Request: In Konqueror's session restore dialog, allow user to chose which sessions to restore

Dawit Alemayehu adawit at
Mon Oct 1 21:56:54 BST 2012

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 1, 2012, 8:56 p.m.)

Review request for KDE Base Apps and David Faure.


- Changed the code that shows the session dialog into its own KDialog instance.
- Changed the top level entry item to be checkable so that it can be checked or unchecked when the user select or unselect all the session entries underneath it.
- Updated the rest of the code accordingly.


The attached patch fixes one of those pet peeve bugs that infurate me from time to time by allowing me to unselect the sessions I do not want to be restored when Konqueror's restore session dialog pops up.

This addresses bug 260282.

Diffs (updated)

  konqueror/src/konqsessionmanager.cpp 68a003f 
  konqueror/src/konqsessionmanager.h ee629e4 



* Unselected sessions should not be restored.
* If all available sessions are selected (the default), the behavior should remain the same as it is today.
* If all available sessions are unselected, disable the "Restore Session" button.

Screenshots (updated)

old restore dialog
new restore dialog
new restore dialog v2
new restore dialog v3


Dawit Alemayehu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list