[dolphin] [Bug 413564] Add save tabs feature to Dolphin

Nate Graham bugzilla_noreply at kde.org
Wed Nov 20 14:26:27 GMT 2019


https://bugs.kde.org/show_bug.cgi?id=413564

--- Comment #3 from Nate Graham <nate at kde.org> ---
Git commit 46e5b32ed9293749661eb5a5a8c32e77b1853f1e by Nate Graham.
Committed on 20/11/2019 at 14:26.
Pushed by ngraham into branch 'master'.

Also allow invoking session restoration logic when apps are manually launched

Summary:
Right now session restoration logic only ever gets invoked when apps are
started
automatically after rebooting, because of the check for
`qApp->isSessionRestored()`.
This limits the feature to only working for that use case, since
`qApp->isSessionRestored()`
is set by Qt and we can't override it in the app itself to signal that we want
the session
restoration behavior for other reasons. As a result, we can't use it to
implement the
behavior of restoring the condition of app when it's manually launched, which
has been
requested for several of our apps.

This patch removes the check for `qApp->isSessionRestored()`, which opens the
door to
implementing the requested features in our apps.
Related: bug 397463

Test Plan:
D11382 now works

Normal session restoration behavior after a reboot is unchanged

Apps with session restoration behavior but without a user-facing option to
invoke it
don't restore session at inappropriate times.

Reviewers: davidedmundson, #frameworks

Reviewed By: davidedmundson

Subscribers: kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D25106

M  +0    -6    src/kmainwindow.cpp

https://commits.kde.org/kxmlgui/46e5b32ed9293749661eb5a5a8c32e77b1853f1e

-- 
You are receiving this mail because:
You are on the CC list for the bug.



More information about the kfm-devel mailing list