D22067: Don't clear the session KConfing object too early

[Ahmad Samir]

ahmadsamir added a comment.


  Looking closer at this, it's more complicated than I initially thought :)
  
  The location inside a sidebar, persistence and visibility settings of a tool view are only read from the config in Sidebar::restoreSession(); so at best with the create-toolview-on-demand nature of the projects plugin, we can only set the saved position setting. I'd say that's OK, because this diff here is more about the anonymous session, which is a special case.
  
  If the user has a named-saved session, most of the time if he has files from a git/svn/hg repo open, he's probably not going to close them before saving the session or quitting kate.
  
  So, just restoring the position setting would be OK?
  
  Incidentally, why is the projects plugin has its tool views created on demand? that point doesn't seem to be documented (hint: you should document it :)).

REPOSITORY
  R40 Kate

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

To: ahmadsamir, #kate, cullmann, dhaumann
Cc: kwrite-devel, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190626/38a7c7f0/attachment.html>