<table><tr><td style="">mglb created this revision.<br />Restricted Application added a project: Konsole.<br />Restricted Application added a subscriber: konsole-devel.<br />mglb requested review of this revision.
</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/D14037">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When the focus is in <tt style="background: #ebebeb; font-size: 13px;">IncrementalSearchBar</tt>, unhandled keys (i.e. those<br />
not handled by text input box or <tt style="background: #ebebeb; font-size: 13px;">unhandledMovementKeyPressed</tt> signal)<br />
are sent to a program running in the terminal, which is probably not<br />
what an user expects. At least Up, Down, and Tab keys are propagated.</p>

<p>This patch disables propagation. Shortcuts, movement keys, and keys<br />
handled by search bar still works.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Run <tt style="background: #ebebeb; font-size: 13px;">cat</tt></li>
<li class="remarkup-list-item">Open search box</li>
<li class="remarkup-list-item">Press Up/Down/Tab key</li>
</ul>

<p>Actual result: keys are sent to <tt style="background: #ebebeb; font-size: 13px;">cat</tt><br />
Expected result: nothing should happen (implemented)<br />
Alternative expected result: focus should move from text input to first<br />
button</p>

<p>Regression test:</p>

<p>Following keys/shortcuts should work:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Shift+Up/Down/PgUp/PgDown - terminal contents scrolling</li>
<li class="remarkup-list-item">Ctrl+Shift+T - open new tab (unless it was changed)</li>
<li class="remarkup-list-item">Enter - search next</li>
<li class="remarkup-list-item">Esc - close search box</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>arc/IncrementalSearchBox-do-not-propagate-unhandled-keys-to-parent (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14037">https://phabricator.kde.org/D14037</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/IncrementalSearchBar.cpp</div></div></div><br /><div><strong>To: </strong>mglb<br /><strong>Cc: </strong>konsole-devel, herrold, ngraham, maximilianocuria, hindenburg<br /></div>