<table><tr><td style="">mglb updated this revision to Diff 30608.<br />mglb edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-au4bdhwdrtmzani/">(Show Details)</a><br />mglb 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/D11668">View Revision</a></tr></table><br /><div><div><p>Fix null pointer dereference; minor fixes</p></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>  * <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Scroll the view with Shift+Up/Down/PgUp/PgDown/Home/End - the view should scroll</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Hold down URL hint key - the URL should be underlined with square and a digit on the left<br />
  * Press <URL-hint-key>+1 - the URL should open in a web browser</span><br />
  <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Hold down URL hint key - the URL should be underlined with square and a digit on the left</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Scroll the view with Shift+Up/Down/PgUp/PgDown/Home/End - the view should scroll</span><br />
  <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Press <URL-hint-key>+1 - the URL should open in a web browser</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Split the view, do previous test in each split - only currently active split should scroll</span><br />
* Regressions tests<div style="padding: 8px 0;">...</div>  * Press Ctrl+S - the time should not stop updating<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
  * Use IME to enter text - nothing in terminal should be shown</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D11668?vs=30510&id=30608">https://phabricator.kde.org/D11668?vs=30510&id=30608</a></div></div><br /><div><strong>BRANCH</strong><div><div>arc/allow-view-controlling-keys-in-read-only-mode</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11668">https://phabricator.kde.org/D11668</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/SessionController.cpp<br />
src/TerminalDisplay.cpp<br />
src/Vt102Emulation.cpp</div></div></div><br /><div><strong>To: </strong>mglb, Konsole, hindenburg<br /><strong>Cc: </strong>hindenburg, Konsole, herrold, ngraham<br /></div>