D20518: Enable multi-tab-only shortcuts after restoring a multi-tab session from session management

Andreas Hartmetz noreply at phabricator.kde.org
Sat Apr 13 22:46:11 BST 2019


ahartmetz created this revision.
ahartmetz added reviewers: tcanabrava, Konsole.
Herald added a project: Konsole.
Herald added a subscriber: konsole-devel.
ahartmetz requested review of this revision.

REVISION SUMMARY
  The container's addView() method emits a signal viewAdded() that enables multi-tab QActions when the second tab is added. In fact, it seems a bit unnecessary that almost all calls of createView() are followed by addView()? There might be some cleanup potential there.
  
  This patch is supposed to be a suggestion and base for discussion for just mentioned reasons.

REPOSITORY
  R319 Konsole

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

AFFECTED FILES
  src/ViewManager.cpp

To: ahartmetz, tcanabrava, #konsole
Cc: konsole-devel, gennad, thsurrel, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20190413/aaa6e1dc/attachment.html>


More information about the konsole-devel mailing list