<table><tr><td style="">hindenburg edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-wairf7jrr52rozl/">(Show Details)</a><br />hindenburg edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-wkmcjun6zjsfjr7/">(Show Details)</a>
</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/D11763">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">* Do not send mouse events and emulated up/down keys to programs when<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
  </span> read-only mode is enab<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">led</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">led.<br />
* Set the currentTerminalDisplay before sending key events, like in<br />
   keyPressEvent</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Set the currentTerminalDisplay before sending key events, like in keyPressEvent.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* wheelEvent code is restructured a bit to easily separate code not</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* wheelEvent code is restructured a bit to easily separate code not</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">  </span> allowed in read-only mode.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>* Actual result: scrolling works in read only mode (but only when no key<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> <br />
  </span> has been pressed before on 2nd terminal screen)<br />
<br />
* Start konsole<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
* Run a program which uses mouse events, e.g. `vim` <br />
   (enable mouse with `:set mouse=a`)</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Run a</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Make the</span> program <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">which uses</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">state suitable for using</span> mouse events, e.g. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`vim` (enable mouse with `:set mouse=a`)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">enter a few</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Make the program state suitable for using mouse events, e.g. enter a few</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">  </span> lines of text<div style="padding: 8px 0;">...</div>  * Click and move (selecting)<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
* Expected result: mouse events are not passed to the program in read-only</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Expected result: mouse events are not passed to the program in read-only</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">  </span> mode; nothing happens<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>arc/properly-handle-mouse-in-read-only-mode</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11763">https://phabricator.kde.org/D11763</a></div></div><br /><div><strong>To: </strong>mglb, Konsole, hindenburg<br /><strong>Cc: </strong>hindenburg, Konsole, herrold, ngraham<br /></div>