D15720: [TabBox] Fix Arrow Key / Keyboard Events in QML Alt+Tab Skins

davidedmundson added a comment.


  Both old and patched code seem weird.
  
  I would expect to send it to window->activeFocusItem
  (or just send the event to the window which will then do that)
  
  It will then match what a normal QML user would expect.
  That /should/ work for at least the main breeze theme (untested)
  
  > @mart: Should I add main mainItem->setParent(contentItem());, or should we fix this in KWin?
  
  Not having mainItem a visual child of the content item is super weird - but changing anything in Dialog is risky.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D15720

To: Zren, #kwin
Cc: davidedmundson, graesslin, mart, ngraham, zzag, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol