<table><tr><td style="">jnoack added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10862" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D10862#218315" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D10862#218315</a>, <a href="https://phabricator.kde.org/p/hindenburg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@hindenburg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Thanks, overall I don't see any major issues.</p>

<p>There are some issues below which I don't think should hold up committing this.  Perhaps later you could work on them if you're interested.</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">dbus methods still work on locked tabs: example qdbus $KONSOLE_DBUS_SERVICE /Sessions/3 sendText hi</li>
<li class="remarkup-list-item">Certain menus should be disabled on locked tabs: (right click menu) Switch Profile, * Scrollback, Rename(?). Split View..... and more...</li>
</ol></div>
</blockquote>

<p>Yeah, I can take a look at them next weekend I think.</p>

<p>I will update the diff regarding your comments. Should be good to go then.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10862#inline-53267" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">hindenburg</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ViewContainer.cpp:307</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Why remove this action when the menu closes?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The action is added dynamically in openTabContextMenu from the respective sessionController for the underlying tab. If the action is not removed again, the context menu will keep all the actions from all different controllers.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10862" rel="noreferrer">https://phabricator.kde.org/D10862</a></div></div><br /><div><strong>To: </strong>jnoack, Konsole, hindenburg<br /><strong>Cc: </strong>hindenburg, ngraham, Konsole<br /></div>