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

[Christoph Cullmann]

cullmann added a comment.


  Taking a look at the code, I think one can not just keep that pointer/group around that long.
  
  One could try to e.g. remember the keys with values for positions in finishRestore() and use that as overwrite in createToolView like we do there with
  
    // try the restore config to figure out real pos
    if (m_restoreConfig && m_restoreConfig->hasGroup(m_restoreGroup)) {
        KConfigGroup cg(m_restoreConfig, m_restoreGroup);
        pos = (KMultiTabBar::KMultiTabBarPosition) cg.readEntry(QStringLiteral("Kate-MDI-ToolView-%1-Position").arg(identifier), int(pos));
    }
  
  As long as the config pointer is valid, no operation showing/moving toolviews will have any effect.

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/20190624/1d9cee4f/attachment-0001.html>