D11382: Add an option to show tabs from last time when Dolphin starts

David Edmundson noreply at phabricator.kde.org
Tue Oct 29 01:52:31 GMT 2019


davidedmundson added a comment.


  It's a shame that we're re-using the session restore internals, but aren't calling KMainWindow::restore to do it all properly with just the one code path. I feel like it should be do-able with just a tiny change.

INLINE COMMENTS

> dolphinmainwindow.cpp:203
> +    // other Dolphin windows open
> +    if (GeneralSettings::rememberOpenedTabs()) {
> +        // Check for existing Dolphin windows in a separate nested if() to

you need to check you're not being loaded as part of a session restore otherwise you'll have two paths call readProperties

qApp->isSessionRestored()

REPOSITORY
  R318 Dolphin

BRANCH
  arcpatch-D11382

REVISION DETAIL
  https://phabricator.kde.org/D11382

To: ngraham, #dolphin, #vdg, feverfew, meven, elvisangelaccio, ndavis
Cc: davidedmundson, ndavis, intika, feverfew, kfm-devel, ngraham, broulik, #dolphin, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20191029/1779c0a0/attachment.htm>


More information about the kfm-devel mailing list