<table><tr><td style="">abika accepted this revision as: abika.<br />abika 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/D26291">View Revision</a></tr></table><br /><div><div><p>Good improvement in my opinion!<br />
My test results:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The CTRL+SHIFT+UP/DOWN shortcuts work perfectly. Only when unfocusing the navigation bar a second time, the edit mode stays on. It might be a widget problem, but i consider it minor and it happened also before this patch.</li>
<li class="remarkup-list-item">Then i set CTRL+UP/DOWN as alternative shortcuts for the same actions to see if this can be used for backwards compatibility: This does not work when trying to unfocusing the navigation bar, the embedded terminal or the command line. All these widgets are capturing the key press and the shortcut action is not activated.</li>
</ul>

<p>So we have to consider that CTRL+UP/DOWN cannot be used anymore as before and users may complain about it. However, this did also not work for the embedded terminal (see bug report) and for the sidebar before this patch.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26291">https://phabricator.kde.org/D26291</a></div></div><br /><div><strong>To: </strong>gengisdave, Krusader, yurchor, abika<br /><strong>Cc: </strong>abika, asensi, yurchor, kde-doc-english, Krusader, gennad, fbampaloukas, nmel, mjanczara, miroslavm, skadinna, janlepper, martinkostolny, gengisdave, andreaska<br /></div>