<table><tr><td style="">cblack 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/D29486">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D29486#665078" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D29486#665078</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Can you expand on why it's broken currently?</p></div>
</blockquote>

<p>Not sure if you're asking "why" on a "why is this an issue" or "why is this happening in the first place" so I'll answer both.</p>

<p>Why old behaviour is an issue:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Normally, menus allow you to interact with them via the keyboard. Menus with panels as a transient parent don't allow you to do this, and the patch fixes that lack of expected behaviour.</li>
</ul>

<p>Why this happens in the first place:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Normally, QMenus don't grab keyboard focus themselves, and instead the parent window takes keyboard events and uses it to update the state of the QMenu. Since the Plasmashell panel cannot normally grab keyboard focus, they're unable to use keyboard events to update the state of the QMenu. Fortunately, the QMenu responds fine if you send the keyboard events directly to it, so that's what this patch causes KWin to do.</li>
</ul></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/D29486">https://phabricator.kde.org/D29486</a></div></div><br /><div><strong>To: </strong>cblack, KWin<br /><strong>Cc: </strong>davidedmundson, 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>