<table><tr><td style="">mglb created this revision.<br />mglb added a reviewer: Konsole.<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/D13081">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The key (the part before modifiers) is interpreted by QKeySequence.<br />
For special keys (everything except printable characters), it uses names<br />
from <tt style="background: #ebebeb; font-size: 13px;">Qt::Key</tt> without <tt style="background: #ebebeb; font-size: 13px;">Key_</tt> prefix. For character keys, it uses the<br />
character itself, and it is not possible to use e.g. <tt style="background: #ebebeb; font-size: 13px;">Comma</tt>. For this<br />
reason limited set of characters limits possible key bindings.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">In profile's key bindings settings (or your keysym file) create entry:</li>
</ul>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">key ,-Shift+Ctrl-Alt+Ansi : "Hello from keysym"</pre></div>

<p>Change <tt style="background: #ebebeb; font-size: 13px;">-Alt</tt> to <tt style="background: #ebebeb; font-size: 13px;">+Alt</tt> if you have a conflict. Do not change shift<br />
(the shortcut would trigger on '<' instead of ',')</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Save and close settings</li>
<li class="remarkup-list-item">Press <tt style="background: #ebebeb; font-size: 13px;">Ctrl+,</tt></li>
</ul>

<p>Expected result: "Hello from keysym" is typed<br />
Actual result:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">the key binding does not work</li>
<li class="remarkup-list-item">empty rows appear in key bindings table</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>wip/Allow-any-characters-as-a-key-in-key-binding (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13081">https://phabricator.kde.org/D13081</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/KeyboardTranslator.cpp</div></div></div><br /><div><strong>To: </strong>mglb, Konsole<br /><strong>Cc: </strong>konsole-devel, Konsole, herrold, ngraham, maximilianocuria, hindenburg<br /></div>