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