<table><tr><td style="">davidedmundson 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/D29272">View Revision</a></tr></table><br /><div><div><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);">"The Wayland compositor is however under no obligation to disable
 all of its shortcuts, and may keep some special key combo for its own
 use, including but not limited to one allowing the user to forcibly
 restore normal keyboard events routing in the case of an unwilling
 client. The compositor may also use the same key combo to reactivate
 an existing shortcut inhibitor that was previously deactivated on
 user request."</pre></div>

<p>Do we think it's important to have this when this lands or just deal with it if it becomes an issue?</p>

<p>I think it should be relatively easy, just an earlier filter that calls:<br />
waylandServer()->keyboardShortcutsInhibitManager()->getShortcutsInhibitor(surface, seat)->setActive(false);</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29272#inline-167351">View Inline</a><span style="color: #4b4d51; font-weight: bold;">input.cpp:792</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span style="color: #aa4000">auto</span> <span class="n">seat</span> <span style="color: #aa2211">=</span> <span class="n">waylandServer</span><span class="p">()</span><span style="color: #aa2211">-></span><span class="n">seat</span><span class="p">();</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span style="color: #aa4000">auto</span> <span class="n">surface</span> <span style="color: #aa2211">=</span> <span class="n">seat</span><span style="color: #aa2211">-></span><span class="n">focusedKeyboardSurface</span><span class="p">();</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span style="color: #aa4000">auto</span> <span class="n">inhibitor</span> <span style="color: #aa2211">=</span> <span class="n">waylandServer</span><span class="p">()</span><span style="color: #aa2211">-></span><span class="n">keyboardShortcutsInhibitManager</span><span class="p">()</span><span style="color: #aa2211">-></span><span class="n">getShortcutsInhibitor</span><span class="p">(</span><span class="n">surface</span><span class="p">,</span> <span class="n">seat</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Surface can be null</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29272">https://phabricator.kde.org/D29272</a></div></div><br /><div><strong>To: </strong>bport, davidedmundson, zzag<br /><strong>Cc: </strong>kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>